-
Notifications
You must be signed in to change notification settings - Fork 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
Edit release-process.md for clarity #2161
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.
See comments.
doc/release-process.md
Outdated
@@ -53,7 +53,9 @@ In `configure.ac` and `clientversion.h`: | |||
If this release changes the behavior of the protocol or fixes a serious bug, we may | |||
also wish to change the `PROTOCOL_VERSION` in `version.h`. | |||
|
|||
Build and commit to update versions, and then perform the following command: | |||
Build, then commit these changes to update versions. |
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.
Needs to be "commit then build" so that the built binary doesn't include -dirty
.
doc/release-process.md
Outdated
@@ -101,8 +103,9 @@ Run `./fetch-params.sh`. | |||
|
|||
## D. Make tag for the newly merged result | |||
|
|||
In this example, we ensure master is up to date with the | |||
previous merged PR, then: | |||
Checkout master and pull the latest version to ensure master is up to date with the release PR which has recently been reviewed and merged. Check the last commit on the local and remote versions of master to make sure they are the 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.
s/which has recently been reviewed and merged/merged in B4/
- Break the line?
2ba99e3
to
8144a21
Compare
@str4d incorporated your comments |
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.
ACK
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.
ACK
@zkbot r+ |
📌 Commit 8144a21 has been approved by |
Edit release-process.md for clarity For #2160
☀️ Test successful - zcash |
For #2160