Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.4k
Docs: Improved phrasing#14069
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Docs: Improved phrasing#14069
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Removed usage of second person pronouns in the section and made the assumption of "uneasiness" in code style transition more neutral.
the-knights-who-say-ni commentedJun 14, 2019
Hello, and thanks for your contribution! I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed thePSF contributor agreement (CLA). Unfortunately we couldn't find an account corresponding to your GitHub username onbugs.python.org (b.p.o) to verify you have signed the CLA (this might be simply due to a missing "GitHub Name" entry in your b.p.o account settings). This is necessary for legal reasons before we can look at your contribution. Please followthe steps outlined in the CPython devguide to rectify this issue. You cancheck yourself to see if the CLA has been received. Thanks again for your contribution, we look forward to reviewing it! |
aeros commentedJun 14, 2019
Just created an account an bugs.python.org (made sure to specify github account name) and signed the PSF. Are the other components required for minor changes in the docs, or is that solely for code modifications? |
willingc commentedJun 15, 2019
Hi @aeros167. Thanks for the PR. It looks as if the tests are failing on the documentation build due to trailing whitespace in line 34. Please review and update the PR. When the tests all pass and the CLA status icon changes to signed (may take a few days), feel free to @-mention me and I'll take another look at the PR. |
aeros commentedJun 21, 2019 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@willingc The CLA status is now approved. I hadn't realized that I needed to perform the CLA check manually in order for it to be updated on existing pull requests. Good to know for other projects which may have similar prerequisites. I also removed the white space at the end of line 34. The only tests that aren't passing now are the labeling related tests, which likely wouldn't be relevant for small doc changes. Thanks for letting me know. For future pull requests, was this one appropriately sized? I tried to avoid making it too small by improving parts of the section instead of a single sentence, but I didn't want to make changes across the entire doc for my first pull request to the project. I have some experience with writing/editing technical documentation, but I'm entirely new with contributing to large open source projects on GitHub. I've heard that smaller commits are generally preferable. |
willingc commentedJun 21, 2019
Thanks @aeros167 for the corrections and the PR. You are correct that smaller pull requests are generally favored. As an FYI thedevguide has helpful sections on documentation and the development process. I encourage you to look at documentation issues on bugs.python.org as we would appreciate help with those issues. ☀️ |
* master: (599 commits) Docs: Improved phrasing (pythonGH-14069) Remove redundant if check from optional argument function in argparse. (pythonGH-8766) bpo-37289: Add a test for if with ifexpr in the peephole optimiser to detect regressions (pythonGH-14127) Update What's New in Python 3.9 (pythonGH-14253) bpo-36511: Improve ARM32 buildbot scripts (pythonGH-14251) bpo-37151: remove _PyCFunction_FastCallDict (pythonGH-14269) Fix typo, 'widger' -> 'widget', in idlelib/tree.py (pythonGH-14263) Fix bpo number in News file. (pythonGH-14260) bpo-37342: Fix the incorrect nb_index's type in typeobj documentation (pythonGH-14241) Update What's New in Python 3.8 (pythonGH-14239) bpo-36710: Use tstate in pylifecycle.c (pythonGH-14249) Add missing single quote in io.TextIOWrapper.reconfigure documentation (pythonGH-14246) bpo-36511: Add buildbot scripts and fix tests for Windows ARM32 buildbot (pythonGH-13454) bpo-37333: Ensure IncludeTkinter has a value (pythonGH-14240) bpo-37331: Clarify format of socket handler messages in the documentation. (pythonGH-14234) bpo-37258: Not a bug, but added a unit test and updated documentation. (pythonGH-14229) bpo-36710: Remove PyImport_Cleanup() function (pythonGH-14221) Fix name of '\0'. (pythonGH-14222) bpo-36710: Add tstate parameter in import.c (pythonGH-14218) Document typing.ForwardRef (pythonGH-14216) ...
* Docs: Improved phrasing Removed usage of second person pronouns in the section and made the assumption of "uneasiness" in code style transition more neutral.* Removed trailing whitespace on line 34
* Docs: Improved phrasing Removed usage of second person pronouns in the section and made the assumption of "uneasiness" in code style transition more neutral.* Removed trailing whitespace on line 34
Summary: Removed usage of second person pronouns in the coding style section and made the assumption of "uneasiness" in transition between styles come across more neutrally. Also slightly improved the wording.