-
Notifications
You must be signed in to change notification settings - Fork 60
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
Migrate to Python3 #25
Conversation
@@ -9,7 +9,7 @@ | |||
class ViewModel: | |||
@staticmethod | |||
def is_correct(color): | |||
return map(operator.methodcaller("isdigit"), color) == [True] * 3 | |||
return list(map(operator.methodcaller("isdigit"), color)) == [True] * 3 |
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.
the whole line looks suspicious, why not return all([isdigit(x) for x in color])
or something similar?
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.
@okainov well, this is your line, i've just fixed it a bit ;-) Feel free to submit a PR, will be able to return to it a bit later...
.travis.yml
Outdated
python: | ||
- "2.7" | ||
# python: | ||
# - "2.7" | ||
#- "3.5" |
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.
Probably makes sense to update to 3.8 :)
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.
So far relying on whatever TravisCI suggests out of the box. Will need to revisit later with all thee versions for all dependencies...
No description provided.