Skip to content

Conversation

@poteto
Copy link
Member

@poteto poteto commented Feb 27, 2025

Fixes up a few things in the script and workflow to make it possible to run in CI without interactive prompts.

@react-sizebot
Copy link

react-sizebot commented Feb 27, 2025

Comparing: 227e841...6a8a199

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.js = 6.68 kB 6.68 kB +0.05% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 515.73 kB 515.73 kB = 92.10 kB 92.10 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 565.65 kB 565.65 kB = 100.87 kB 100.87 kB
facebook-www/ReactDOM-prod.classic.js = 636.71 kB 636.71 kB = 112.07 kB 112.07 kB
facebook-www/ReactDOM-prod.modern.js = 627.03 kB 627.03 kB = 110.49 kB 110.49 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 6404952

for (let i = 0; i < entries.length; i++) {
const [bestGuessVersion, packages] = entries[i];
const packageNames = packages.map(name => theme.package(name)).join(', ');
if (ci !== true) {
Copy link
Member Author

Choose a reason for hiding this comment

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

Only change here was this

Comment on lines +19 to +26
if (ci === true) {
if (publishVersion != null) {
versionsMap.set(packageName, publishVersion);
} else {
const {major, minor, patch} = semver(version);

// Guess the next version by incrementing patch.
// The script will confirm this later.
// By default, new releases from mains should increment the minor version number,
// and patch releases should be done from branches.
if (branch === 'main') {
versionsMap.set(packageName, `${major}.${minor + 1}.0`);
console.error(
'When running in CI mode, a publishVersion must be supplied'
);
process.exit(1);
Copy link
Member Author

Choose a reason for hiding this comment

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

Only change here was this

Fixes up a few things in the script and workflow to make it possible to run in CI without interactive prompts.
@poteto poteto merged commit 4c9392b into main Feb 27, 2025
196 checks passed
@poteto poteto deleted the pr32488 branch March 10, 2025 21:33
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