Skip to content

Conversation

Stormheg
Copy link
Member

Includes changes from #62

See discussion in #51


This PR implements many of the changes needed to move the package to Wagtail Nest. The package itself was already transfered to the Wagtail Nest organization in December 2024 by Springload. I now have found the time to make the remaining changes necessary to make it all official.

Thanks to @haydngreatnews from Springload for his help to make the transfer happen.

I've made a bunch of changes to make the package more 'modern' for the year 2025.

  • Convert from setup.py to pyproject.toml
  • Use pre-commit
  • Use ruff instead of flake8 and isort. The config is based on the one now being used for wagtail.org, see Switch to ruff for linting wagtail/wagtail.org#488
  • Use PyPI Trusted Publishing to automate publishing new releases. I've already set up the trusted publisher on the PyPI project page.
  • Update the tox configuration; drop and add currently supported Wagtail and Django versions
  • Set up nightly tests. Test the package against the latest development versions of Wagtail, Django and django-recaptcha to catch incompatibilities early.

nickmoreton and others added 26 commits November 14, 2024 15:09
And update the supported Django and Wagtail versions and classifiers.
I set the dependency to wagtail>=5.0 becuase thats what the testing is run against.
And implement flake8 and isort rules through ruff.
- Ruff format
- End of line fixes
- Pretty JSON
>  error: Multiple top-level packages discovered in a flat-layout: ['wagtailcaptcha', 'coverage_html_report'].
Rely on pre-commit to run linters and formatters
Drop unsupported versions of Wagtail and Django, add new ones
Copy link

codecov bot commented Feb 20, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Can't set it up - not available for open source anymore? Signup now
requires an invite.
@Stormheg Stormheg marked this pull request as ready for review February 20, 2025 15:24
@Stormheg
Copy link
Member Author

Not sure what this is. Hope it goes away once the new CI setup is merged.

afbeelding

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