-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
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
- The selected value must not be orphaned so it may be useful to set the value two blocks back from the tip. | ||
- 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
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...
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 👍
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
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
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: