-
Notifications
You must be signed in to change notification settings - Fork 1.2k
docs: update release process document to follow-up v20 release #5710
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
docs: update release process document to follow-up v20 release #5710
Conversation
64cf5e6 to
ed9d86e
Compare
|
Link to preview how this MD file is rendered by GitHub: https://github.com/dashpay/dash/blob/ed9d86e9805e93a05fe6b81a2acb764e34b9ffb5/doc/release-process.md |
doc/release-process.md
Outdated
| - Testnet should be set some tens of thousands back from the tip due to reorgs there. | ||
| - This update should be reviewed with a reindex-chainstate with assumevalid=0 to catch any defect | ||
| that causes rejection of blocks in the past history. | ||
| * [ ] Ensure that governance is functional after sentinel changes are done |
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.
this is not needed moving forward
| * [ ] Ensure that governance is functional after sentinel changes are done |
doc/release-process.md
Outdated
| * [ ] Pasta Guix Build | ||
| * [ ] UdjinM6 Guix Build | ||
| * [ ] Thephez Guix Build | ||
| * [ ] kwvg Guix Build | ||
| * [ ] knst Guix Build |
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'd prefer not enumerating these here as this can change from version to version
doc/release-process.md
Outdated
| * [ ] Pasta | ||
| * [ ] UdjinM6 | ||
| * [ ] Thephez | ||
| * [ ] kwvg | ||
| * [ ] knst |
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.
same
thephez
left a comment
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.
Some suggested updates and corrections...
thephez
left a comment
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.
Looks good. Thanks for updating this 👍
UdjinM6
left a comment
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.
pls see UdjinM6@1038c1a for some suggestions
Co-authored-by: thephez <thephez@users.noreply.github.com>
f1e92c3 to
d029fdb
Compare
UdjinM6
left a comment
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.
utACK
PastaPastaPasta
left a comment
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.
utACK for squash merge
Issue being fixed or feature implemented
Our Release Process document is not exactly matched with our template issue that created by copy-paste from previous release.
For the next release just copy test from this document to new issue
What was done?
Release Process document is updated to match with our real release process.
This document has also detailed instructions for many steps (in compare to the issue that we use now which is more checklist) to make releasing process easier for all participant.
How Has This Been Tested?
Tested on air by 2 last released: v19, v20: #5694
Breaking Changes
N/A
Checklist: