Skip to content

ci: add support for django 3.2 #61

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

Merged
merged 1 commit into from
Sep 14, 2021

Conversation

mariajgrimaldi
Copy link
Contributor

Description
This PR supports Django 3.2 in CI as the Open edX community effort to give Django 3.2 support to all its dependencies. Here's the issue if you're interested.

@mariajgrimaldi
Copy link
Contributor Author

mariajgrimaldi commented Sep 10, 2021

Hi there @NarenderRajuB @marksweb @vinitkumar!
I'm tagging you in this issue as you're the latest contributors to the repo! Can you help me move this forward?

Copy link
Member

@marksweb marksweb left a comment

Choose a reason for hiding this comment

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

Added a couple of minor requests to help the test suite going forwards.

@codecov-commenter
Copy link

Codecov Report

Merging #61 (b628fbe) into master (5ad9c1f) will not change coverage.
The diff coverage is n/a.

❗ Current head b628fbe differs from pull request most recent head b6f6b95. Consider uploading reports for the commit b6f6b95 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master      #61   +/-   ##
=======================================
  Coverage   99.63%   99.63%           
=======================================
  Files           9        9           
  Lines         541      541           
  Branches       89       89           
=======================================
  Hits          539      539           
  Misses          1        1           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5ad9c1f...b6f6b95. Read the comment docs.

@mariajgrimaldi
Copy link
Contributor Author

Hey! @marksweb I followed your suggestions! Thank you

@marksweb marksweb merged commit 4273a72 into django-cms:master Sep 14, 2021
@marksweb
Copy link
Member

@mariajgrimaldi fantastic, thank your for your contribution.

@mariajgrimaldi
Copy link
Contributor Author

Great! Thank you @marksweb
I'm looking forward to the new release! :)

@marksweb
Copy link
Member

@mariajgrimaldi I can't actually do releases but I've raised with the team the fact that this needs a 3.2 compatible release following these changes 👍

@mariajgrimaldi mariajgrimaldi mentioned this pull request Oct 7, 2021
@natabene
Copy link

@marksweb Thanks so much for looking into this. Have you had any news from the team? Do you know what the ETA for the 3.2 compatible release is?

@NicolaiRidani
Copy link
Contributor

Hi @natabene @mariajgrimaldi I just wanted to let you know that our contributor team is looking into this. You can find us in Slack in #release-management (you are welcome to join us by the way). I hope it will be released as soon as possible. But we still have to clarify a few things internally first.

About us: The django-cms project (and all other projects that are part of the django CMS Github organization) is officially supported by the non-profit django CMS Association. We are a group of volunteers (individuals, agencies, companies, universities, etc.) who want to keep django-cms (and its ecosystem) alive and take it to the next level. The project(s) is/are entirely community driven. So we rely on the power of the community. Everyone is welcome to join us. For more information, feel free to contact me at nicolai@django-cms.org or join us on Slack (link above). Thank you :)

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.

5 participants