fix: expand Python version compatibility#351
Open
kiram15 wants to merge 4 commits intoopenedx:mainfrom
Open
Conversation
9934f69 to
4741229
Compare
fcf480b to
eb7c661
Compare
67bf976 to
590c66e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds the flexibility to support Python 3.11 as well as 3.12 to unblock work to transition all enterprise code into well-defined plugin hooks.
Motivation:
Right now, the repo works well with both python 3.11 and 3.12 (link), but the constraint in the setup file deems that only versions past 3.12 are allowed.
This blocks all of the work that the Titans team (enterprise team at 2u) is doing to deploy new plugins, as they have not yet migrated to 3.12 in edx/edx-platform. This in turn is slowing down the work of removing the hard dependency of the platform code in the edx-enterprise repos.
There is no specific functionality that is being used from python 3.12, so this would unblock that initiative.
Supporting information
To speed this up, some maintainers were pinged via slack, but they are also asked as reviewers here.
The proposal here is to allow installers to use in 3.11 at their own risk. CI testing would remain using 3.12 only.
Testing instructions
It is possible to run the test suite using 3.11.
Deadline
Soon if possible to speed up the work of removing enterprise.
Checklists
Check off if complete or not applicable:
Merge Checklist:
Post Merge: