8000 Wagtail 64 upgrade by nickmoreton · Pull Request #29 · torchbox/wagtail-makeup · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Wagtail 64 upgrade #29

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Mar 20, 2025
Merged

Wagtail 64 upgrade #29

merged 5 commits into from
Mar 20, 2025

Conversation

nickmoreton
Copy link
Collaborator
@nickmoreton nickmoreton commented Mar 18, 2025 •
edited
Loading

This pull request includes several updates to dependencies and configurations, as well as improvements to the development and testing setup.

Dependency updates:

  • Updated black to version 25.1.0, isort to version 6.0.1, and flake8 to version 7.1.2.

Development and testing setup:

  • Added a new development_extras list with black, flake8, and isort dependencies, and modified extras_require to include development.
  • Added support for Python 3.13 and Django 5.1, and updated Wagtail versions to include 6.3 and 6.4. Removed support for Python 3.8, Django 3.2, and Wagtail 6.0.

Documentation updates:

  • Updated the pip install command to include the development extras.
  • Changelog updated

Actions updates:

  • Remove python 3.8 and add python 3.13

@nickmoreton
Add tox testing for python 3.13, Wagtail 6.3 and 6.4, Django 5.1
9d2de04 
Also adds development_testing dependencies so that the pre-commit hooks
can be run without installing the packages globally.
@nickmoreton
Add extra dependencies for development
f5967ba
@nickmoreton
Remove python 3.8, add python 3.13 to the test matrix
a3f77f1
@nickmoreton nickmoreton self-assigned this Mar 18, 2025
@codecov-commenter
Copy link
codecov-commenter commented Mar 18, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
+ 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.

🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nickmoreton nickmoreton mentioned this pull request Mar 18, 2025
Closed
@nickmoreton nickmoreton marked this pull request as ready for review March 18, 2025 15:26
@ianmeigh
Copy link

Suggestion: The setup config file needs updating to match the new minimum requirements.

@ianmeigh
Copy link

Looks good @nickmoreton, I've left one suggestion.

@nickmoreton nickmoreton merged commit 9e5085b into torchbox:main Mar 20, 2025
5 checks passed
@nickmoreton nickmoreton deleted the support/wagtail-64-upgrade branch March 20, 2025 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nickmoreton nickmoreton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nickmoreton @codecov-commenter @ianmeigh
0