Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

doc: release-time commits #60

Merged
merged 1 commit into from
Apr 17, 2015
Merged

doc: release-time commits #60

merged 1 commit into from
Apr 17, 2015

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Apr 15, 2015

Add allowance for release time commits and error corrections.
Per: #48 (comment)
Example: nodejs/node#1421 and nodejs/node@fd90b33...50e9fc1.

Add allowance for release time commits and error corrections.
Per: #48 (comment)
Example: nodejs/node#1421 and nodejs/node@fd90b33...50e9fc1.
@jasnell
Copy link
Member Author

jasnell commented Apr 15, 2015

@Fishrock123 @rvagg

@Fishrock123
Copy link
Contributor

👍

@rvagg
Copy link
Member

rvagg commented Apr 15, 2015

1.7.0 & 1.7.1 is an interesting example of this (not the only one), there was a commit that only broke the build on releases so we didn't notice until I tried to make a 1.7.0, but I'd already tagged so had to bail on 1.7.0 entirely and skip to 1.7.1 for the fix.

jasnell added a commit that referenced this pull request Apr 17, 2015
@jasnell jasnell merged commit ef8ac78 into master Apr 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants