-
Notifications
You must be signed in to change notification settings - Fork 49
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
build: drop support for python 3.8 and 3.9 #1792
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Question: should we bump the gt4py version in this PR? And if yes, should it a major or a minor version bump? |
I would not change the version here, as we typically changed version before release. Let's discuss if we should change that, e.g. something like gridtools cpp where |
@@ -292,7 +288,8 @@ docstring-code-format = true | |||
# NPY: NumPy-specific rules | |||
# RUF: Ruff-specific rules | |||
ignore = [ | |||
'E501' # [line-too-long] | |||
'E501', # [line-too-long] | |||
'B905' # [zip-without-explicit-strict] # TODO(egparedes): Reevaluate this rule |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd vote for removing this in a next PR.
This PR only changes configuration files to drop support for old python versions. Refactorings and code changes will follow in different PRs.