Skip to content

Conversation

@TheAhumMaitra
Copy link

I improved the editor.py, which is now more readable and attractive.

@bedevere-app
Copy link

bedevere-app bot commented Aug 24, 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.

@picnixz
Copy link
Member

picnixz commented Aug 24, 2025

Please stop creating PRs that are purely cosmetics as we don't accept them.

@picnixz picnixz closed this Aug 24, 2025
@terryjreedy
Copy link
Member

Ahum, each PR must be created as a separate branch from main so you do not get commits from other changes mixed in.

This looks like you ran editor.py through black.py, an independent 3rd party python code formatter that in some respects violates the spirit and letter of PEP 8. A few of the changes I would make if editing a function, but perhaps 95% of the changes, in particular ' to " and reformats of calls, are to PEP 8 compliant code that is intentionally the way it is. So I would reject them.

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