-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
doc: document current release procedure #2099
Conversation
@@ -91,7 +96,7 @@ The _CHANGELOG.md_ entry should take the following form: | |||
* Include the full list of commits since the last release here | |||
``` | |||
|
|||
### 5. Update _src/node_version.h_ | |||
### 4. Update _src/node_version.h_ |
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 know this is a tiny thing, but we should probably make a script for it. I know I've definitely messed this up in subtle ways before.
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.
agreed, and I messed up the tag comment for 2.3.3 today as well, this could either be Jenkinsised or put into a script but I don't see an obvious path yet
so much feedback, thanks folks, pushed a new version, ptal |
_TODO: Update doc with announce procedure when we figure this out._ | ||
The iojs.org website will automatically rebuild and include the new version. You simply need to announce the build, preferably via twitter with a message such as: | ||
|
||
> v2.3.2 of @official_iojs is out @ https://iojs.org/dist/latest/ changelog @ https://github.com/nodejs/io.js/blob/master/CHANGELOG.md#2015-07-01-version-232-rvagg … something here about notable changes |
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.
latest/ changelog
has two spaces.
LGTM minus one nit |
PR-URL: nodejs#2099 Author: Jeremiah Senkpiel <fishrock123@rocketmail.com>
84b3915 merged, thanks folks |
PR-URL: nodejs#2099 Author: Jeremiah Senkpiel <fishrock123@rocketmail.com>
/R=@Fishrock123 @chrisdickinson