Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Checks for Wagtail 6.3 #28

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

nickmoreton
Copy link
Collaborator

This pull request includes several updates to dependencies, Python version support, and testing configurations.

The commits from: #27 which also included #26 so they can be closed now.

Python version support:

  • Removed support for Python 3.8 in .github/workflows/tests.yml, tox.ini, and setup.cfg. [1] [2] [3]

Dependency updates:

  • Updated black, isort, and flake8 versions in .pre-commit-config.yaml.
  • Updated setuptools and pip versions in setup.cfg.
  • Updated coverage version in setup.py.
  • Updated python-unsplash version in setup.cfg and setup.py. [1] [2]

Django and Wagtail support:

  • Added support for Django 5.1 and Wagtail 6.x in tox.ini and CHANGELOG.md. [1] [2]

Miscellaneous:

  • Removed SiteMiddleware from wagtailmakeup/tests/settings.py.

This was referenced Nov 21, 2024
@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.68%. Comparing base (bce562a) to head (ea71f80).
Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   73.33%   73.68%   +0.35%     
==========================================
  Files           5        5              
  Lines          75       76       +1     
==========================================
+ Hits           55       56       +1     
  Misses         20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants