Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on May 25, 2022. It is now read-only.

Added the project for the issue #198 #205

Merged
AdityaJ7 merged 5 commits into Python-World:master from leader2one:javokhir
Sep 8, 2020

Conversation

Copy link
Contributor

@leader2one leader2one commented Sep 8, 2020
edited by AdityaJ7
Loading

Description

Number Guessing Game

Fixes #198

Type of change

  • New feature

Checklist:

Please tick all the boxes that are fulfilled by your Pull request

  • I have named my files and folder according to this project's guidelines

  • My code follows the style guidelines of this project

  • I have commented my code, particularly in hard-to-understand areas

  • I have created a helpful and easy to understand README.md

  • I have included a requirements.txt file(if external libraries are required)

  • My changes dont produce any warnings.

  • I have added a working sample/screenshot of the script

Copy link
Member

AdityaJ7 commented Sep 8, 2020
edited
Loading

@leader2one LGTM, Awesome Job. You can also add the number of guesses it took the user to win and increase the range of numbers to make it more challenging, eg:1 to 100.

Copy link
Contributor Author

@leader2one LGTM, Awesome Job. You can also add the number of guesses it took the user to win and increase the range of numbers to make it more challenging, eg:1 to 100.

thanks. Also I placed an example working screenshot in readme file. Is image being showed ?

AdityaJ7 reacted with thumbs up emoji

Copy link
Member

AdityaJ7 commented Sep 8, 2020

@leader2one LGTM, Awesome Job. You can also add the number of guesses it took the user to win and increase the range of numbers to make it more challenging, eg:1 to 100.

thanks. Also I placed an example working screenshot in readme file. Is image being showed ?

No, it wasn't showing But I fixed it.

Copy link
Member

@tusharnankani tusharnankani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the changes! Keep it up! @leader2one 👍🏻

leader2one and chavarera reacted with thumbs up emoji
Copy link
Member

@chavarera chavarera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

validate your code with PEP8 standards

Check PEP8 Validation
flake8 main.py

Suggestions:

main.py:1:14: W291 trailing whitespace
main.py:2:1: W293 blank line contains whitespace
main.py:3:30: W291 trailing whitespace
main.py:4:1: W293 blank line contains whitespace
main.py:5:35: W291 trailing whitespace
main.py:6:27: W291 trailing whitespace
main.py:7:30: W291 trailing whitespace
main.py:8:1: W293 blank line contains whitespace
main.py:9:32: W291 trailing whitespace
main.py:10:34: W291 trailing whitespace
main.py:11:36: W291 trailing whitespace
main.py:12:32: W291 trailing whitespace
main.py:13:16: W291 trailing whitespace
main.py:15:1: W293 blank line contains whitespace
main.py:16:43: W291 trailing whitespace
main.py:17:1: W293 blank line contains whitespace
main.py:18:33: W291 trailing whitespace
main.py:19:13: W291 trailing whitespace
main.py:21:1: W293 blank line contains whitespace
main.py:22:36: W291 trailing whitespace
main.py:23:25: W291 trailing whitespace
main.py:24:1: W293 blank line contains whitespace
main.py:25:38: W291 trailing whitespace
main.py:26:36: W291 trailing whitespace
main.py:27:24: W291 trailing whitespace
main.py:28:1: W293 blank line contains whitespace
main.py:29:36: W291 trailing whitespace
main.py:30:35: W291 trailing whitespace
main.py:31:42: W291 trailing whitespace
main.py:32:37: W291 trailing whitespace
main.py:33:36: W291 trailing whitespace
main.py:34:80: E501 line too long (94 > 79 characters)
main.py:35:9: E265 block comment should start with '# '
main.py:37:1: W293 blank line contains whitespace
main.py:38:32: W291 trailing whitespace
main.py:39:29: W291 trailing whitespace
main.py:40:27: W291 trailing whitespace
main.py:41:25: W291 trailing whitespace
main.py:42:75: W291 trailing whitespace
main.py:43:1: W293 blank line contains whitespace
main.py:44:33: W291 trailing whitespace
main.py:45:33: W291 trailing whitespace
main.py:46:13: W291 trailing whitespace
main.py:47:10: W291 trailing whitespace
main.py:48:75: W291 trailing whitespace
main.py:49:1: W293 blank line contains whitespace
main.py:50:40: W291 trailing whitespace
main.py:52:1: W391 blank line at end of file

Copy link
Contributor Author

validate your code with PEP8 standards

Check PEP8 Validation
flake8 main.py

Suggestions:

main.py:1:14: W291 trailing whitespace
main.py:2:1: W293 blank line contains whitespace
main.py:3:30: W291 trailing whitespace
main.py:4:1: W293 blank line contains whitespace
main.py:5:35: W291 trailing whitespace
main.py:6:27: W291 trailing whitespace
main.py:7:30: W291 trailing whitespace
main.py:8:1: W293 blank line contains whitespace
main.py:9:32: W291 trailing whitespace
main.py:10:34: W291 trailing whitespace
main.py:11:36: W291 trailing whitespace
main.py:12:32: W291 trailing whitespace
main.py:13:16: W291 trailing whitespace
main.py:15:1: W293 blank line contains whitespace
main.py:16:43: W291 trailing whitespace
main.py:17:1: W293 blank line contains whitespace
main.py:18:33: W291 trailing whitespace
main.py:19:13: W291 trailing whitespace
main.py:21:1: W293 blank line contains whitespace
main.py:22:36: W291 trailing whitespace
main.py:23:25: W291 trailing whitespace
main.py:24:1: W293 blank line contains whitespace
main.py:25:38: W291 trailing whitespace
main.py:26:36: W291 trailing whitespace
main.py:27:24: W291 trailing whitespace
main.py:28:1: W293 blank line contains whitespace
main.py:29:36: W291 trailing whitespace
main.py:30:35: W291 trailing whitespace
main.py:31:42: W291 trailing whitespace
main.py:32:37: W291 trailing whitespace
main.py:33:36: W291 trailing whitespace
main.py:34:80: E501 line too long (94 > 79 characters)
main.py:35:9: E265 block comment should start with '# '
main.py:37:1: W293 blank line contains whitespace
main.py:38:32: W291 trailing whitespace
main.py:39:29: W291 trailing whitespace
main.py:40:27: W291 trailing whitespace
main.py:41:25: W291 trailing whitespace
main.py:42:75: W291 trailing whitespace
main.py:43:1: W293 blank line contains whitespace
main.py:44:33: W291 trailing whitespace
main.py:45:33: W291 trailing whitespace
main.py:46:13: W291 trailing whitespace
main.py:47:10: W291 trailing whitespace
main.py:48:75: W291 trailing whitespace
main.py:49:1: W293 blank line contains whitespace
main.py:50:40: W291 trailing whitespace
main.py:52:1: W391 blank line at end of file

sorry i do not understand. can you clearify more? or give hint?

Copy link
Member

validate your code with PEP8 standards
Check PEP8 Validation
flake8 main.py
Suggestions:

main.py:1:14: W291 trailing whitespace
main.py:2:1: W293 blank line contains whitespace
main.py:3:30: W291 trailing whitespace
main.py:4:1: W293 blank line contains whitespace
main.py:5:35: W291 trailing whitespace
main.py:6:27: W291 trailing whitespace
main.py:7:30: W291 trailing whitespace
main.py:8:1: W293 blank line contains whitespace
main.py:9:32: W291 trailing whitespace
main.py:10:34: W291 trailing whitespace
main.py:11:36: W291 trailing whitespace
main.py:12:32: W291 trailing whitespace
main.py:13:16: W291 trailing whitespace
main.py:15:1: W293 blank line contains whitespace
main.py:16:43: W291 trailing whitespace
main.py:17:1: W293 blank line contains whitespace
main.py:18:33: W291 trailing whitespace
main.py:19:13: W291 trailing whitespace
main.py:21:1: W293 blank line contains whitespace
main.py:22:36: W291 trailing whitespace
main.py:23:25: W291 trailing whitespace
main.py:24:1: W293 blank line contains whitespace
main.py:25:38: W291 trailing whitespace
main.py:26:36: W291 trailing whitespace
main.py:27:24: W291 trailing whitespace
main.py:28:1: W293 blank line contains whitespace
main.py:29:36: W291 trailing whitespace
main.py:30:35: W291 trailing whitespace
main.py:31:42: W291 trailing whitespace
main.py:32:37: W291 trailing whitespace
main.py:33:36: W291 trailing whitespace
main.py:34:80: E501 line too long (94 > 79 characters)
main.py:35:9: E265 block comment should start with '# '
main.py:37:1: W293 blank line contains whitespace
main.py:38:32: W291 trailing whitespace
main.py:39:29: W291 trailing whitespace
main.py:40:27: W291 trailing whitespace
main.py:41:25: W291 trailing whitespace
main.py:42:75: W291 trailing whitespace
main.py:43:1: W293 blank line contains whitespace
main.py:44:33: W291 trailing whitespace
main.py:45:33: W291 trailing whitespace
main.py:46:13: W291 trailing whitespace
main.py:47:10: W291 trailing whitespace
main.py:48:75: W291 trailing whitespace
main.py:49:1: W293 blank line contains whitespace
main.py:50:40: W291 trailing whitespace
main.py:52:1: W391 blank line at end of file

sorry i do not understand. can you clearify more? or give hint?

ok @leader2one PEP8(Python Enhancement Proposals) set some guideline while writing your code.
these guideline are the set of rules to follow while coding

style guide lines to write python code read more here..
flake8 is an external module helps to identify and validate these set of rules

Install flake8

pip install flake8

Run flake8

flake8 main.py 

Copy link
Contributor Author

validate your code with PEP8 standards
Check PEP8 Validation
flake8 main.py
Suggestions:

main.py:1:14: W291 trailing whitespace
main.py:2:1: W293 blank line contains whitespace
main.py:3:30: W291 trailing whitespace
main.py:4:1: W293 blank line contains whitespace
main.py:5:35: W291 trailing whitespace
main.py:6:27: W291 trailing whitespace
main.py:7:30: W291 trailing whitespace
main.py:8:1: W293 blank line contains whitespace
main.py:9:32: W291 trailing whitespace
main.py:10:34: W291 trailing whitespace
main.py:11:36: W291 trailing whitespace
main.py:12:32: W291 trailing whitespace
main.py:13:16: W291 trailing whitespace
main.py:15:1: W293 blank line contains whitespace
main.py:16:43: W291 trailing whitespace
main.py:17:1: W293 blank line contains whitespace
main.py:18:33: W291 trailing whitespace
main.py:19:13: W291 trailing whitespace
main.py:21:1: W293 blank line contains whitespace
main.py:22:36: W291 trailing whitespace
main.py:23:25: W291 trailing whitespace
main.py:24:1: W293 blank line contains whitespace
main.py:25:38: W291 trailing whitespace
main.py:26:36: W291 trailing whitespace
main.py:27:24: W291 trailing whitespace
main.py:28:1: W293 blank line contains whitespace
main.py:29:36: W291 trailing whitespace
main.py:30:35: W291 trailing whitespace
main.py:31:42: W291 trailing whitespace
main.py:32:37: W291 trailing whitespace
main.py:33:36: W291 trailing whitespace
main.py:34:80: E501 line too long (94 > 79 characters)
main.py:35:9: E265 block comment should start with '# '
main.py:37:1: W293 blank line contains whitespace
main.py:38:32: W291 trailing whitespace
main.py:39:29: W291 trailing whitespace
main.py:40:27: W291 trailing whitespace
main.py:41:25: W291 trailing whitespace
main.py:42:75: W291 trailing whitespace
main.py:43:1: W293 blank line contains whitespace
main.py:44:33: W291 trailing whitespace
main.py:45:33: W291 trailing whitespace
main.py:46:13: W291 trailing whitespace
main.py:47:10: W291 trailing whitespace
main.py:48:75: W291 trailing whitespace
main.py:49:1: W293 blank line contains whitespace
main.py:50:40: W291 trailing whitespace
main.py:52:1: W391 blank line at end of file

sorry i do not understand. can you clearify more? or give hint?

ok @leader2one PEP8(Python Enhancement Proposals) set some guideline while writing your code.
these guideline are the set of rules to follow while coding

style guide lines to write python code read more here..
flake8 is an external module helps to identify and validate these set of rules

Install flake8

pip install flake8

Run flake8

flake8 main.py 

ok i know what pep8 is. but i am thinking about what to do with flake8 main.py. If i run this locally, there will be no change in the code in this repo. What exacty i should do right now? thanks for supporting

Copy link
Contributor Author

validate your code with PEP8 standards
Check PEP8 Validation
flake8 main.py
Suggestions:

main.py:1:14: W291 trailing whitespace
main.py:2:1: W293 blank line contains whitespace
main.py:3:30: W291 trailing whitespace
main.py:4:1: W293 blank line contains whitespace
main.py:5:35: W291 trailing whitespace
main.py:6:27: W291 trailing whitespace
main.py:7:30: W291 trailing whitespace
main.py:8:1: W293 blank line contains whitespace
main.py:9:32: W291 trailing whitespace
main.py:10:34: W291 trailing whitespace
main.py:11:36: W291 trailing whitespace
main.py:12:32: W291 trailing whitespace
main.py:13:16: W291 trailing whitespace
main.py:15:1: W293 blank line contains whitespace
main.py:16:43: W291 trailing whitespace
main.py:17:1: W293 blank line contains whitespace
main.py:18:33: W291 trailing whitespace
main.py:19:13: W291 trailing whitespace
main.py:21:1: W293 blank line contains whitespace
main.py:22:36: W291 trailing whitespace
main.py:23:25: W291 trailing whitespace
main.py:24:1: W293 blank line contains whitespace
main.py:25:38: W291 trailing whitespace
main.py:26:36: W291 trailing whitespace
main.py:27:24: W291 trailing whitespace
main.py:28:1: W293 blank line contains whitespace
main.py:29:36: W291 trailing whitespace
main.py:30:35: W291 trailing whitespace
main.py:31:42: W291 trailing whitespace
main.py:32:37: W291 trailing whitespace
main.py:33:36: W291 trailing whitespace
main.py:34:80: E501 line too long (94 > 79 characters)
main.py:35:9: E265 block comment should start with '# '
main.py:37:1: W293 blank line contains whitespace
main.py:38:32: W291 trailing whitespace
main.py:39:29: W291 trailing whitespace
main.py:40:27: W291 trailing whitespace
main.py:41:25: W291 trailing whitespace
main.py:42:75: W291 trailing whitespace
main.py:43:1: W293 blank line contains whitespace
main.py:44:33: W291 trailing whitespace
main.py:45:33: W291 trailing whitespace
main.py:46:13: W291 trailing whitespace
main.py:47:10: W291 trailing whitespace
main.py:48:75: W291 trailing whitespace
main.py:49:1: W293 blank line contains whitespace
main.py:50:40: W291 trailing whitespace
main.py:52:1: W391 blank line at end of file

sorry i do not understand. can you clearify more? or give hint?

ok @leader2one PEP8(Python Enhancement Proposals) set some guideline while writing your code.
these guideline are the set of rules to follow while coding

style guide lines to write python code read more here..
flake8 is an external module helps to identify and validate these set of rules

Install flake8

pip install flake8

Run flake8

flake8 main.py 

ok gimme a time i 'll solve this issue in the least time

Copy link
Member

here will be no change in th

ok 👍

First validate your code in your local 
sync the main repository 
commit your changes in local
then push your code to forked repo
and update your PR

Copy link
Contributor Author

leader2one commented Sep 8, 2020
edited
Loading

here will be no change in th

ok 👍

First validate your code in your local 
sync the main repository 
commit your changes in local
then push your code to forked repo
and update your PR

thanks man. Today i learned good stuffs because of you. I solved this issue with autopep8 . You may check the updated version

AdityaJ7 and chavarera reacted with thumbs up emoji tusharnankani reacted with hooray emoji

Copy link
Member

@AdityaJ7 AdityaJ7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is fixed now, good job !!!

leader2one reacted with thumbs up emoji
Copy link
Member

@chavarera chavarera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good to go @leader2one

leader2one reacted with thumbs up emoji
@AdityaJ7 AdityaJ7 merged commit f5cdcf8 into Python-World:master Sep 8, 2020
@leader2one leader2one deleted the javokhir branch September 8, 2020 15:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
3 more reviewers

@chavarera chavarera chavarera approved these changes

@AdityaJ7 AdityaJ7 AdityaJ7 approved these changes

@tusharnankani tusharnankani tusharnankani approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Number guessing game

AltStyle によって変換されたページ (->オリジナル) /