Skip to content

[ci] Don't erroneously mark failures as successes#32493

Merged
poteto merged 1 commit into
mainfrom
pr32493
Feb 28, 2025
Merged

[ci] Don't erroneously mark failures as successes#32493
poteto merged 1 commit into
mainfrom
pr32493

Conversation

@poteto
Copy link
Copy Markdown
Collaborator

@poteto poteto commented Feb 28, 2025

Randomly noticed this when I looked at a recent DevTools regression test run.

I don't recall why we added continue-on-error previously, but I believe it was to keep all jobs in the matrix running even if one were to fail, in order to fully identify any failures from code changes like build or test failures.

There is now a fail-fast option which does this. continue-on-error now means:

Prevents a workflow run from failing when a job fails. Set to true to allow a workflow run to pass when this job fails.

so it's not correct to use it.

Randomly noticed this when I looked at a recent [DevTools regression test run](https://github.com/facebook/react/actions/runs/13578385011).

I don't recall why we added `continue-on-error` previously, but I believe it was to keep all jobs in the matrix running even if one were to fail, in order to fully identify any failures from code changes like build or test failures.

There is now a `fail-fast` option which does this. [`continue-on-error`](https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idcontinue-on-error) now means:

> Prevents a workflow run from failing when a job fails. Set to true to allow a workflow run to pass when this job fails.

so it's not correct to use it.
@react-sizebot
Copy link
Copy Markdown

Comparing: ca12911d1fbe755b9b2b7d1bf548589317311a82...c208cf94e935d2fb0f18264f292776f82b1d721d

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.11% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 518.24 kB 518.24 kB = 92.43 kB 92.43 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.11% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 570.57 kB 570.57 kB = 101.56 kB 101.56 kB
facebook-www/ReactDOM-prod.classic.js = 638.06 kB 638.06 kB = 112.28 kB 112.28 kB
facebook-www/ReactDOM-prod.modern.js = 628.38 kB 628.38 kB = 110.70 kB 110.70 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against fc07675

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.

3 participants