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

feat: drop Python 3.8 and add Python 3.12 support #514

Conversation

Faraz32123
Copy link

@Faraz32123 Faraz32123 commented Nov 14, 2024

This PR includes the below items:

  • Dropped support for Python 3.8 and added support for Python 3.12.
  • Removed importlib-metadata<7 pin from constraints as it's unpin issue was closed.
  • Updated pylintrc file with latest version of edx-lint.
  • Fixed app_label error for model named CourseAllowPIISharingInLTIFlag.
  • Released a new version.
  • close Drop Python 3.8 & Add Support for Python 3.12 #501

Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

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

One thing to fix but then I think this is good for me to merge/release.

importlib-metadata<7
# Adding pin to avoid any major upgrade
# Issue for unpinning: https://github.com/openedx/edx-platform/issues/35265
pymongo<4.4.1
Copy link
Contributor

Choose a reason for hiding this comment

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

The only reason this repo depends on pymongo is because of opaque-keys. You should let that repo set any restrictions on pymongo for our testing and not constrain it here.

Copy link
Author

Choose a reason for hiding this comment

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

sounds good! I have removed the pymongo constraint here.

- Dropped support for Python 3.8 and added support for Python 3.12.
- Removed importlib-metadata<7 pin from constraints as it's unpin issue was closed.
- Updated pylintrc file with latest version of edx-lint.
- Fixed app_label error for model named CourseAllowPIISharingInLTIFlag.
- Released a new version.
@Faraz32123 Faraz32123 force-pushed the faraz/feat-drop_python3.8_and_add_support_for_python_3.12 branch from 0db0dac to 5beb2e1 Compare November 14, 2024 15:05
@feanil feanil merged commit 70aa7c2 into openedx:master Nov 14, 2024
6 checks passed
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.

Drop Python 3.8 & Add Support for Python 3.12
2 participants