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

Update tox to remove unsupported Django, add more Wagtail, and add django-recaptcha #60

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

haydngreatnews
Copy link
Collaborator

@haydngreatnews haydngreatnews commented Sep 15, 2024

We were a few releases behind on Django (5.0+) and Wagtail (6.0+), so we're adding these to the matrix, and also retiring some older out-of-support versions.

Additionally, with the (not-that) recent changes to django_recaptcha, we add these to the matrix as a variable, so that if we want to vary that in future, we can.

@haydngreatnews haydngreatnews force-pushed the chore/more-tox branch 2 times, most recently from 527478e to 7879f1c Compare September 16, 2024 00:43
@haydngreatnews haydngreatnews marked this pull request as ready for review September 16, 2024 01:28
@gdelavil
Copy link

@haydngreatnews are the dependabot alerts misconfigured in this project? Nothing's coming up, which is odd.

@haydngreatnews
Copy link
Collaborator Author

haydngreatnews commented Sep 16, 2024

Pass, but its never going to bring anything up against our one declared dependency which doesn't have any advisories

Copy link

@gdelavil gdelavil left a comment

Choose a reason for hiding this comment

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

@haydngreatnews looks good, however it would be good if you added a short description of what motivated this PR.

@haydngreatnews haydngreatnews merged commit d9f3484 into main Sep 19, 2024
8 checks passed
@haydngreatnews haydngreatnews deleted the chore/more-tox branch September 19, 2024 03:58
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