Skip to content

Conversation

@bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Dec 4, 2018

Based on team discussion during today's sync, the next version number will be:

  • A minor bump if releasing from master
  • A patch bump if releasing from a branch

The release script still enables user overrides for these values. This only affects the suggested next version.


if (!existsSync(join(cwd, 'build'))) {
await exec(`mkdir ./build`, {cwd});
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change (and the one in check-out-packages) was just fixing an edge case I noticed while testing these changes.

@bvaughn
Copy link
Contributor Author

bvaughn commented Dec 7, 2018

This seems uncontroversial enough that I'm going to just land it.

@bvaughn bvaughn merged commit ce43a8c into facebook:master Dec 7, 2018
@bvaughn bvaughn deleted the better-suggested-version-numbers branch December 7, 2018 17:08
jetoneza pushed a commit to jetoneza/react that referenced this pull request Jan 23, 2019
n8schloss pushed a commit to n8schloss/react that referenced this pull request Jan 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants