Skip to content
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

Misc small adjustments to build scripts #20723

Merged
merged 1 commit into from
Feb 3, 2021

Conversation

bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Feb 3, 2021

  • Remove --build param from release scripts.
  • Update README instructions to match the current state of the build scripts.
  • Re-add ability to create reproducible builds to DevTools download script to support publishing to the Firefox store.

Follow up to #20720

Also update instructions to match recent script changes.

Also add reproducible commit SHA to post download instructions to support publishing the Firefox DevTools extension.
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Feb 3, 2021
@bvaughn
Copy link
Contributor Author

bvaughn commented Feb 3, 2021

I think I ran through all of the various scripts and the README to make sure things now match up and work.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 3, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3203821:

Sandbox Source
React Configuration

@sizebot
Copy link

sizebot commented Feb 3, 2021

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 3203821

@sizebot
Copy link

sizebot commented Feb 3, 2021

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 3203821

@bvaughn bvaughn changed the title Remove --build flag from release scripts Misc small adjustments to build scripts Feb 3, 2021
// Otherwise fall back to parsing from the package version number.
// This is important to make the build reproducible (e.g. by Mozilla reviewers).
const buildInfoJSON = join(
cwd,
Copy link
Collaborator

Choose a reason for hiding this comment

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

'Once 4783999 lands you can alternatively read from build2/COMMIT_SHA

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh nice. Fair.

@bvaughn bvaughn merged commit 2d02575 into facebook:master Feb 3, 2021
@bvaughn bvaughn deleted the scripts-remove-build-flag branch February 3, 2021 16:12
koto pushed a commit to koto/react that referenced this pull request Jun 15, 2021
Also update instructions to match recent script changes.

Also add reproducible commit SHA to post download instructions to support publishing the Firefox DevTools extension.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants