Skip to content

Conversation

@bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Aug 24, 2021

  1. Add README documenting new release scripts
  2. Split release script into separate prepare, build, and publish scripts.

Prepare script

devtools-release-prepare.mp4

Build and test script

If you've run the prepare script already:

devtools-release-build.mp4

If you forgot to run the prepare script:

devtools-release-build-no.mp4

Publish script

If you've run the build and test script already:

devtools-release-publish.mp4

If you forgot to run the build and test script:

devtools-release-publish-no.mp4

If the publish script failed or timed out in the middle, you may need to restart it. In that case it's expected that some packages have already been published. If this is ever not expected though, the script will stop:

devtools-release-publish-already.mp4

1. Add README
2. Split release script into separate prepare, build, and publish scripts.
@sizebot
Copy link

sizebot commented Aug 24, 2021

Comparing: 5828580...00a4d9e

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 127.58 kB 127.58 kB = 40.72 kB 40.72 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 130.40 kB 130.40 kB = 41.65 kB 41.65 kB
facebook-www/ReactDOM-prod.classic.js = 405.16 kB 405.16 kB = 75.04 kB 75.04 kB
facebook-www/ReactDOM-prod.modern.js = 393.72 kB 393.72 kB = 73.32 kB 73.32 kB
facebook-www/ReactDOMForked-prod.classic.js = 405.16 kB 405.16 kB = 75.04 kB 75.04 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 00a4d9e

@bvaughn bvaughn merged commit b996468 into facebook:main Aug 24, 2021
@bvaughn bvaughn deleted the devtools-release-script branch August 24, 2021 22:59
@jstejada
Copy link
Contributor

this is awesome!! should we also update our wiki?

@bvaughn
Copy link
Contributor Author

bvaughn commented Aug 26, 2021

Thanks!

Yeah, we should. Maybe we can do this as part of the next release that one of you do?

zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
@kambyzkargrshhamt
Copy link

Good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Component: Developer Tools 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.

6 participants