-
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
#208 support HackerRank again #297
Conversation
NOTE: the problem is not fixed yet
f-string は Python 3.6 から、そうだった |
@fukatani 確認たのみます |
tests/command_download_hackerrank.py
Outdated
def snippet_call_download(self, *args, **kwargs): | ||
tests.command_download.snippet_call_download(self, *args, **kwargs) | ||
|
||
# def test_call_download_hackerrank_beautiful_array(self): |
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.
このコメントアウトはTODOですか?
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.
TODOです。
以前は動いていたのですが、実装方法を変えたので動かなくなりました。
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.
1件だけコメントつけましたが、それ以外はいいと思います。
コメントしてもらったところ、単にコメントアウトだけなのよくないなあと思ったので |
LGTM |
close #208
close #151
復活させました