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 support for python 3.8 #579

Merged

Conversation

marcelotrevisani
Copy link
Member

@marcelotrevisani marcelotrevisani commented Nov 13, 2024

  • Drop support for python 3.8 and 3.9
  • Update type annotations to Python 3.9+ syntax
  • Replace legacy type annotations with Python 3.9+ syntax for improved readability and conciseness.
  • Update the pre-commit hook version for ruff to v0.7.3 for linting purposes.

@marcelotrevisani marcelotrevisani requested a review from a team as a code owner November 13, 2024 07:36
pre-commit-ci bot and others added 5 commits November 13, 2024 07:36
Replace legacy type annotations with Python 3.9+ syntax for improved readability and conciseness. Additionally, update the pre-commit hook version for ruff to v0.7.3 for linting purposes.

Signed-off-by: Marcelo Trevisani <[email protected]>
Set Sphinx workflow to use Python 3.12 explicitly and removed testing for Python 3.9 in the tests workflow.

Signed-off-by: Marcelo Trevisani <[email protected]>
Replaced the type hint for the `__instance` attribute in `CLIConfig` from `"CLIConfig"` to `Self` to improve type checking and readability.

Signed-off-by: Marcelo Trevisani <[email protected]>
Added a fallback mechanism to import 'TypeVar' and define 'Self' if 'Self' is not available in the typing module. This ensures compatibility with older Python versions that do not support 'Self'.

Signed-off-by: Marcelo Trevisani <[email protected]>
@marcelotrevisani marcelotrevisani merged commit 10f13e5 into conda:main Nov 13, 2024
8 checks passed
@marcelotrevisani marcelotrevisani deleted the remove-support-for-python38 branch November 13, 2024 08:19
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.

1 participant