Skip to content

Conversation

@simkimsia
Copy link
Contributor

@simkimsia simkimsia commented Apr 2, 2023

Description

This upgrades the support of Django to 4.2rc1

Also this is similar to #219 but continues to use tox_install.sh.

Fixes #212, #214, #218

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added an appropriate CHANGELOG entry
@simkimsia
Copy link
Contributor Author

simkimsia commented Apr 2, 2023

This is a more targeted fix compared to #219

It solely targets the change to support Django 4.2 and maintains the use of tox_install.sh to instsall dependencies.

Inspired by e6ee7c7 (#207)

@simkimsia simkimsia mentioned this pull request Apr 2, 2023
8 tasks
@simkimsia
Copy link
Contributor Author

@thibaudcolas or @bcdickinson can help to review this PR? Thank you

envlist = py{37,38,39,310,311}-dj{32,40,41,main}, lint
envlist =
py{37,38,39,310}-dj32
py{38,39,310,311}-dj40
Copy link
Member

Choose a reason for hiding this comment

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

it is a good time to drop Django 4.0 from the matrix

Copy link
Member

Choose a reason for hiding this comment

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

👍 I’ll do that and Python 3.7 separately.

; Use pre-releases until stable releases are available.
; dj42: Django==4.2a1
; dj42: Django>=4.2,<4.3
dj42: Django==4.2rc1
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
dj42: Django==4.2rc1
dj42: Django>=4.2,<4.3
@zerolab zerolab changed the title feat: Support Django 4.2rc1 feat: Support Django 4.2 May 9, 2023
Copy link
Member

@thibaudcolas thibaudcolas left a comment

Choose a reason for hiding this comment

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

Looking good to me but I think there’s a bit more to do (at least the Django version in dependencies which is the actual blocker)! Will merge and pick up remaining changes.

envlist = py{37,38,39,310,311}-dj{32,40,41,main}, lint
envlist =
py{37,38,39,310}-dj32
py{38,39,310,311}-dj40
Copy link
Member

Choose a reason for hiding this comment

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

👍 I’ll do that and Python 3.7 separately.

@thibaudcolas thibaudcolas changed the title feat: Support Django 4.2 Add Django 4.2 to the testing matrix Aug 19, 2023
@thibaudcolas thibaudcolas merged commit 3aa1e38 into torchbox:main Aug 19, 2023
@simkimsia
Copy link
Contributor Author

Thank you for this @thibaudcolas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants