[issue30031] Improve queens demo (use argparse and singular form)
New submission from Pavlo Kapyshin: Currently Tools/demo/queens.py: - does manual sys.argv parsing - says “Found 1 solutions” I propose to: 1) use argparse; 2) if q.nfound == 1, use “solution”. I you are ok with this, I’ll make a pull request. -- components: Demos and Tools messages: 291428 nosy: paka priority: normal severity: normal status: open title: Improve queens demo (use argparse and singular form) type: enhancement versions: Python 3.7 ___ Python tracker <http://bugs.python.org/issue30031> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue30031] Improve queens demo (use argparse and singular form)
Changes by Pavlo Kapyshin : -- pull_requests: +1228 ___ Python tracker <http://bugs.python.org/issue30031> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue30031] Improve queens demo (use argparse and singular form)
Pavlo Kapyshin added the comment: Louie Lu, thank you for the review. Different quotes were already used in file, so someone should decide which ones must be used: single or double. Regarding the -n option, I tried to be backwards compatible. Again, a decision is needed. The only place where I had a choice to not do something related to PEP 8 was adding a line between Queens class and main function. I decided to mention that fact in pull request. In other places I had to follow contribution guide (therefore add code that follows style guide). -- ___ Python tracker <http://bugs.python.org/issue30031> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
