-
Notifications
You must be signed in to change notification settings - Fork 48.8k
Promote a canary from NPM to be a stable release #14255
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
Conversation
Also updated Circle CI build script to generate and add a build-info.json file to artifacts
`cp -r ${tempDirectory}/build/node_modules/*.tgz ${cwd}/build/node_modules/` | ||
); | ||
|
||
// Unpack packages and parepare to publish. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo on "parepare"
const {getBuildInfo, handleError} = require('./utils'); | ||
|
||
// This local build script exists for special case, manual builds. | ||
// The typical suggesgted release process is to create a canary from a CI artifact. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo on "suggesgted"
// If the source dependency's version and the constraint match, | ||
// we will need to update the constraint to point at the dependency's new release version, | ||
// (e.g. scheduler@^0.11.0 becomes scheduler@^0.12.0 when we release scheduler 0.12.0). | ||
// Othewise we leave the constraint alone (e.g. react@^16.0.0 doesn't change between releases). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo on "Othewise"
// Update all package JSON versions and their dependencies/peerDependencies. | ||
// This must be done in a way that respects semver constraints (e.g. 16.7.0, ^16.7.0, ^16.0.0). | ||
// To do this, we use the dependencies defined in the source package JSONs, | ||
// because the canary dependencies have already been falttened to an exact match (e.g. 0.0.0-ddaf2b07c). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo on "falttened"
Thanks. I can't type. I'll fix those. |
Typo fixes included in next PR, #14260 |
I'm guessing you didn't actually publish the canary you're trying to promote to stable 😄 You can either run with a |
Already merged via #14260 |
aha that's what it was. |
Issue #14201. Builds on top of #14244 (view the delta).
New script to prepare a canary release to be promoted to stable. This script checks out canary packages from NPM, prompts for new stable versions, and updates the versions (including dependencies) in the checked-out packages.
Learn more about these new release scripts here.