-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update watchers.py #47
Conversation
Added min and max to have the number selected only between 1 and 10
Updated placeholder to limit within input width
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.
I like that.
You'd need to update the relevant test though.. |
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.
Looks good, but tests need updating for it
Sure, thanks for letting me know. |
Updated the test case string From `Enter the guess` to `Guess`
Hi @kkinder , Updated tests. Hope its fine |
Hi @kkinder , Should the documentation also needs corresponding update? |
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.
I have updated the changes
Updated the new changes as per 05-watchers changes based on pull request kkinder/puepy#47
Seems reasonable. |
@kkinder I have updated the documentation also |
Added min and max to have the number selected only between 1 and 10