-
Notifications
You must be signed in to change notification settings - Fork 41
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 - Minor updates #91
Conversation
Hi @katdom13 I think it might be a good time to overhaul some testing dependencies. I tried this locally in
The docs seemed to build OK but there was a configuration warning that might need attention. The tests might fail but there's some clues about that here: #89 |
- name: Install build requirements | ||
run: python -m pip install wheel | ||
- name: Build package | ||
run: python setup.py sdist bdist_wheel | ||
- name: Publish package | ||
uses: pypa/gh-action-pypi-publish@master | ||
uses: pypa/gh-action-pypi-publish@release/v1 |
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.
Will throw an error if published and if the Pypi user has no verified email address.
@kevinhowbrook
Hi @nickmoreton , |
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 Looks good.
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