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/python3.10 support #7126

Merged
merged 25 commits into from Jan 24, 2022
Merged

Conversation

vinitkumar
Copy link
Contributor

@vinitkumar vinitkumar commented Oct 5, 2021

Description

Python3.10 got released last evening. This adds support for Python3.10 in DjangoCMS. Python3.10 is still not available on Github actions to running this tests with 3.10-rc2. Don't merge it yet. Once 3.10 official is out on actions and all tests pass, then we merge it.

Related resources

  • #...
  • #...

Checklist

  • I have opened this pull request against develop
  • I have added or modified the tests when changing logic
  • I have followed the conventional commits guidelines to add meaningful information into the changelog
  • I have read the contribution guidelines and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

@vinitkumar vinitkumar requested a review from marksweb Oct 5, 2021
@vinitkumar vinitkumar marked this pull request as draft Oct 5, 2021
@vinitkumar vinitkumar marked this pull request as ready for review Oct 16, 2021
cms/tests/test_plugins.py Outdated Show resolved Hide resolved
vinitkumar and others added 5 commits Jan 16, 2022
On Python 3.10 django emits a warning (DeprecationWarning) for asyncio
run
- https://stackoverflow.com/questions/70303895/python-3-10-asyncio-gather-shows-deprecationwarning-there-is-no-current-event

We filter out this warning here, since it is not relevant to our test
and still have the tests in place so that I can check for this
behaviour.

- Github Issue: django-cms#7183

Authored-by: Vinit Kumar <vinit.kumar@socialschools.nl>
Signed-off-by: Vinit Kumar <vinit.kumar@socialschools.nl>
Add latest stable version of Python3.10 and also remove Python3.6 since
it EOL'ed in december, 2021

Authored-by: Vinit Kumar <vinit.kumar@socialschools.nl>
Signed-off-by: Vinit Kumar <vinit.kumar@socialschools.nl>
@marksweb
Copy link
Member

@marksweb marksweb commented Jan 17, 2022

I've created a brand new project to test this. Things run, and I can create pages.

But I've done something wrong because I can't open the structure pane 😆

I create new projects so infrequently I always forget what causes this...

Anyway, so far, so good 👍

setup.cfg Show resolved Hide resolved
@@ -8,7 +8,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: [ 3.6, 3.7, 3.8, 3.9, ] # latest release minus three
python-version: [ 3.7, 3.8, 3.9, 3.10.2] # latest release minus three
Copy link
Member

@marksweb marksweb Jan 23, 2022

Choose a reason for hiding this comment

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

If you convert these to strings then '3.10' should ensure we test the latest point release.

python-version: [ '3.7', '3.8', '3.9', '3.10']

And drop the comment maybe?

setup.cfg Outdated

[options]
install_requires =
Django>=2.2
Copy link
Member

@marksweb marksweb Jan 23, 2022

Choose a reason for hiding this comment

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

This should probably match what I did here;
c6c2784

vinitkumar and others added 4 commits Jan 23, 2022
vinitkumar added 3 commits Jan 24, 2022
Authored-by: Vinit Kumar <vinit.kumar@socialschools.nl>
Signed-off-by: Vinit Kumar <vinit.kumar@socialschools.nl>
@marksweb marksweb merged commit 324f085 into django-cms:develop Jan 24, 2022
41 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
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants