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

Django 4.2 support #290

Merged
merged 3 commits into from
Jul 19, 2023
Merged

Django 4.2 support #290

merged 3 commits into from
Jul 19, 2023

Conversation

salman2013
Copy link
Contributor

@salman2013 salman2013 commented Jul 13, 2023

  1. Add Django 4.2 support
  2. Bump the version to '2.4.0'

Ticket ref: edx/upgrades#139

setup.py Outdated
@@ -119,6 +119,7 @@ def is_requirement(line):
'Framework :: Django',
'Framework :: Django :: 3.2',
'Framework :: Django :: 4.0',
Copy link
Contributor

Choose a reason for hiding this comment

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

We probably want to remove 4.0 from the list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed this.

tox.ini Outdated
@@ -19,6 +19,7 @@ setenv =
DJANGO_SETTINGS_MODULE = test_settings
deps =
django32: Django>=3.2,<4.0
django42: Django>=4.2,<4.3
django40: Django>=4.0,<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.

ditto here about removing 4.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed this.

tox.ini Outdated
@@ -1,5 +1,5 @@
[tox]
envlist = py38-django{32,40}-drf{latest},quality,docs
envlist = py38-django{32, 42}-drf{latest},quality,docs
Copy link
Contributor

Choose a reason for hiding this comment

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

Is that extra space ok?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove it.

@salman2013 salman2013 requested a review from nedbat July 18, 2023 11:35
@salman2013 salman2013 merged commit 3201e02 into master Jul 19, 2023
5 checks passed
@salman2013 salman2013 deleted the salman/django-4.2-upgrade branch July 19, 2023 11:54
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.

2 participants