Skip to content

Auto-install (or update) release script Yarn deps before running#11434

Merged
bvaughn merged 1 commit intofacebook:masterfrom
bvaughn:release-script-tweaks
Nov 2, 2017
Merged

Auto-install (or update) release script Yarn deps before running#11434
bvaughn merged 1 commit intofacebook:masterfrom
bvaughn:release-script-tweaks

Conversation

@bvaughn
Copy link
Copy Markdown
Contributor

@bvaughn bvaughn commented Nov 2, 2017

Stepping through a dry-run of the new release process for 16.1.0-beta and realized that there was no guaranteed that we were running the latest version of the release script dependencies.

Copy link
Copy Markdown
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

If you can verify failing Yarn also fails the build, seems good.

@bvaughn
Copy link
Copy Markdown
Contributor Author

bvaughn commented Nov 2, 2017

Yah. Changing the command to yarn foo (to intentionally fail it) halts the build process.

Thanks~

@bvaughn bvaughn merged commit a181ba8 into facebook:master Nov 2, 2017
@bvaughn bvaughn deleted the release-script-tweaks branch November 2, 2017 22:21
@bvaughn bvaughn mentioned this pull request Nov 2, 2017
5 tasks
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.

3 participants