-
Notifications
You must be signed in to change notification settings - Fork 26
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
Bump version numbers and add upgrade guide for the 2.9.0 release #573
Conversation
LGTM, two nits inline. I think keeping the Tails 6 docs is a good idea for now; if we want to keep them a few months longer, it might be worth reorganizing them into their own section instead of copying them with each upgrade guide. But removing them with 2.10.0 is probably fine. |
e724d7d
to
e7e8b54
Compare
Thanks @eloquence, that sounds great. I think we're probably okay to remove them in 2.10.0, but we can decide one way or the other closer to that release once we have a better feel for how widely Tails 6 has been adopted. |
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.
Good to go once release is live.
Thanks! After discussing with @zenmonkeykstop , I added an additional commit which mentions the new Tor PoW option, and links to our guide for enabling it. Will need another review before it can be merged with the release. |
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.
I only reviewed the most recent commit (the Tor PoW notice), and that LGTM. Approving based on Erik's prior approval.
Status
Ready for review
Description of Changes
This PR bumps the version numbers throughout the documentation to 2.9.0, and adds an upgrade guide for this release.
I left the instructions for the Tails 6 upgrade in the guide since some folks may not have completed the upgrade as part of the previous release. If we feel these are not necessary I'm happy to remove them. Just let me know!
Testing
Release
Checklist (Optional)
make docs-lint
) passed locallymake docs-linkcheck
) passedmake docs
) docs at http://localhost:8000