-
Notifications
You must be signed in to change notification settings - Fork 66
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
Wagtail 6.1 #144
Wagtail 6.1 #144
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @katdom13!
Left one question regarding testing in interactive mode.
tox.ini
Outdated
deps = | ||
wagtail>=5.2,<6.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: Is there a reason for uncommenting this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it's just me uncommenting it during testing and forgetting to comment again. Changed this in the newest commit.
Thanks both. As this is purely a tidy up PR, will merge it in its own time |
Closing in favor of #145 |
Notes
Wagtail 6.1 upgrade check list
Code reviewing a consideration.
Use the tick box to indicate a consideration has been code reviewed as OK
What’s new
Changes affecting wagtail customizations
Changes to undocumented internals