Skip to content

Conversation

@cement-hools
Copy link

Improve readability and PEP8 compliance

@ghost
Copy link

ghost commented Mar 14, 2025

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Mar 14, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@skirpichev
Copy link
Contributor

I would guess, code was such by design)

@cement-hools
Copy link
Author

I would guess, code was such by design)

Thank you for your feedback! I understand that the code could have been written this way by design, but I want to point out that following PEP8 improves the readability of the code and makes it more understandable for all developers. This is especially important for the long-term support and development of the project. Many improvements in projects are due to enthusiasts, and I want to help make the project more compliant with modern standards. Such an important piece of code that a beginner encounters must comply with the language standards)

@hauntsaninja
Copy link
Contributor

Thanks, but CPython does not accept PRs that are primarily cosmetic. See also the PEP 8 note about hobgoblins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants