Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

really drop python<=3.7 support #1134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kloczek
Copy link

@kloczek kloczek commented Mar 26, 2024

Filter all code over pyupgrade --py38-plus.

Filter all code over `pyupgrade --py38-plus`.

Signed-off-by: Tomasz Kłoczko <kloczek@github.com>
@lurch
Copy link
Contributor

lurch commented Mar 26, 2024

Are all the vim-comment removals necessary? Whilst I don't use Vim myself, I assume whoever added those headers (@bennuttall or @waveform80 ?) finds them useful; and removing those changes from this PR would make it a bit smaller.

@kloczek
Copy link
Author

kloczek commented Mar 26, 2024

Are all the vim-comment removals necessary?

All python files since python 3.x are UTF-8 and vim already has hardcoded thar in its settings which was not the case with python 2.x.
Nevertheless that patch is 100% result of pyupgrade.

division,
)
str = type('')
str = str
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants