feat: python 3.12 upgrade

[ichintanjoshi]

Description

This PR contains changes for upgrading python 3.8 to python 3.12 while also supporting 3.11.

Changes include dependencies version upgrades

Done as a part of following :-

• Update all Python Libraries to Test on Python 3.11 and 3.12 public-engineering#233
• [edx-platform] Test Python 3.11 edx-platform#34229
• Test user-util on Python 3.11 #25

[openedx-webhooks]

Thanks for the pull request, @ichintanjoshi! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[mphilbrick211]

Hi @ichintanjoshi! Thanks for this contribution! Please let me know if you have any questions regarding submitting a CLA form.

[ichintanjoshi]

Hi @mphilbrick211 I've submitted a CLA yesterday, let me know if there is anything else to be done from my side.

[ichintanjoshi]

Hi @mphilbrick211 CLA process is done.

[feanil]

Rather than doing this here, I think it should be an updated in the relevant requirements/*.in files.

We should definitely add it to the tox.in file which is what's causing the package build to fail.

[ichintanjoshi]

Okay, I'll add and check it accordingly

[feanil]

Also, it looks like part of the issue might be the really old version of tox. It may be worth running make upgrade manually in this repo to get the dependencies up to dot. It may be worth doing that in a separate PR. That might fix it without requiring setuptools manually in tox.in

[ichintanjoshi]

After removing it from here and adding it in tox.in, if I check the run in nektos act, it still gives the same error

[ichintanjoshi]

@feanil after running make upgrade other dependencies are now updated and adding setuptools back to this file the tests are now passing in the act. If I don't keep it in this file it still fails

I took reference from here

[feanil]

As I've done a couple of these, I've realized that we should still be building the requirements using Python 3.8 especially in libraries so that we don't test with newer version of dependencies than an upstream running python 3.8 can handle. We want to be able to run with python 3.8 and 3.11/3.12 using 3.8 dependencies. Can we update this to use 3.8 dependencies?

[ichintanjoshi]

Hi @feanil Okay so I need to create a venv with 3.8 and run make upgrade command ?

Or is it something else ?

[ichintanjoshi]

@feanil this one passes all the tests with 3.12 and doesn't have numpy as well. Should I keep this PR open or close it and create a new one with 3.11 ?

[feanil]

I think you can keep it and just add testing for 3.11 as well, end yes, the make upgrade should still be run with Python 3.8 for now.

[ichintanjoshi]

@feanil added 3.11 part too.

[mphilbrick211]

@feanil - would you mind re-enabling tests to run?

[feanil]

Generally looks good, can you rebase and resolve conflicts and bump the minor version. Then I think this will be good to go.

[feanil]

We should do at least a minor version bump.

Suggested change

	__version__ = '1.0.1'
	__version__ = '1.1.0'

[openedx-webhooks]

@ichintanjoshi Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

[openedx-webhooks]

@ichintanjoshi Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.