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

Remove Python 3.7 (EOL since EOL since 2023-06-27) from CI #1440

Merged
merged 3 commits into from Aug 4, 2023

Conversation

kiendang
Copy link
Collaborator

@kiendang kiendang commented Aug 3, 2023

Copy link
Collaborator

@sjdemartini sjdemartini left a comment

Choose a reason for hiding this comment

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

Cool, LGTM!

One comment: we may want to change the pre-commit pyupgrade arg to use --py38-plus now instead of py37:

args: [--py37-plus]

That could presumably be a separate PR after this, since we'll likely want to re-run it on all existing files to upgrade any syntax that was kept around in support of 3.7.

@kiendang
Copy link
Collaborator Author

kiendang commented Aug 3, 2023

Good catch! We could also take this opportunity to switch to ruff? Ruff implements both pyupgrade and flake8 which we use in our pre-commit.

@sjdemartini
Copy link
Collaborator

Yeah great suggestion. I'm using ruff instead of pyupgrade, flake8, etc in a project of mine, and I'm a big fan!

@kiendang
Copy link
Collaborator Author

kiendang commented Aug 4, 2023

@sjdemartini I just include the change to --py38-plus here since after running pre-commit there's no change.
I was looking into changing to ruff but it's a bit more involved since we need to migrate the flake8 and isort config in setup.cfg so will leave to a future PR.

@firaskafri firaskafri self-requested a review August 4, 2023 08:15
@firaskafri firaskafri merged commit 5eb5fe2 into main Aug 4, 2023
24 checks passed
@firaskafri firaskafri deleted the python3.7-eol branch August 4, 2023 08:15
superlevure pushed a commit to loft-orbital/graphene-django that referenced this pull request Aug 9, 2023
…ython#1440)

* Remove Python 3.7 (EOL since EOL since 2023-06-27) from CI

* Remove unused context

* Use pyupgrade --py38-plus in pre-commit
superlevure pushed a commit to loft-orbital/graphene-django that referenced this pull request Aug 10, 2023
…ython#1440)

* Remove Python 3.7 (EOL since EOL since 2023-06-27) from CI

* Remove unused context

* Use pyupgrade --py38-plus in pre-commit
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