-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug fix of AtCoderContest.from_url() #434
Conversation
For urls like "https://atcoder.jp/contests/abc128/tasks/abc128_a", AtCoderContest.from_url() returns None. This commit will fix it.
新形式の方は問題のurlを引数にするとうまく対応出来ていなかったようです。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なんだか動いてなさそうなので修正お願いします 🙏
URLの新旧の形式によって挙動に差があるのは明らかに混乱を生むので、このプルリクはマージしたいです。 |
Co-Authored-By: Kimiyuki Onaka <[email protected]>
すみません。。.*と$の順番は単純にミスしてました。ありがとうございます。 |
修正できてることが確認できたのでマージします |
@kjnh10 作法的なことに関して、特になにも問題はありませんでした。 一般には、 CONTRIBUTING.md のようなファイルが用意されている場合が多く、もしあればそれに従うとよいです。もちろんなにか問題があってもレビューで指摘されるだけなので、あまり完璧を目指さずにプルリクを投げてしまってもよいと思います。 |
For urls like "https://atcoder.jp/contests/abc128/tasks/abc128_a",
AtCoderContest.from_url() returns None.
This commit will fix it.