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

Updated support for more recent versions of Python and Django #60

Merged
merged 6 commits into from
Dec 9, 2023

Conversation

alanverresen
Copy link
Contributor

@alanverresen alanverresen commented Dec 5, 2023

This is currently still a work in progress, as discussed in #59. Do not merge yet!

I've updated tox and the GitHub action to support more recent versions of Python and Django.
What is currently still planned:

  • add different databases to test matrix
  • add support for Django 5.0

Because services (like postgresql) are not supported yet by act, I'll have to rely more on commits to test GHA workflows from now on.

Changes were based on the following:

- "honeypot" had to be replaced by module's actual name
- a multiline condition with a line break before a binary operator
  triggers flake8's W503; changing that line would trigger another
  error or warning, or more require more changes elsewhere
- removed parts related to testing multiple DB that didn't work
- updated test matrices based on currently supported releases
- simplified GitHub action by relying more on tox
- added a separate job for linter
- forgot to add Python 3.12 to GitHub Actions in previous commit
- the long deprecated method assertEquals of unittest.TestCase was
  removed with the release of Python 3.12; had to be replaced
@alanverresen
Copy link
Contributor Author

I've made all the changes that were planned so far. I also made a temporary repository to freely test the new GitHub Action workflow, so everything should work.

@jamesturk
Copy link
Owner

Thank you, I'll take a closer look at this soon. The one change I'll likely make is to eliminate some of the intermediate versions of Python to save time/compute on test runs. (i.e. only test on Python 3.8 and 3.12 in CI) Thanks so much for taking the time to update this!

@alanverresen
Copy link
Contributor Author

It's my pleasure. I've been using this package for a long time, so when I saw the opportunity to contribute in some way, I felt like it was the right thing to do.

The one change I'll likely make is to eliminate some of the intermediate versions of Python to save time/compute on test runs.

Yes, it's a bit ridiculous right now, but support for Django 4.1 has just ended (December 1, 2023), so it can be removed from the test matrix, and support for Django 3.2 will end April 2024. [1] That should make things more manageable.

[1] https://www.djangoproject.com/download/

@jamesturk jamesturk merged commit 32c9d01 into jamesturk:main Dec 9, 2023
31 checks passed
@alanverresen alanverresen deleted the issue59 branch December 9, 2023 20:59
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.

None yet

2 participants