Skip to content

Commit

Permalink
[Tooling] Update critical artifact list
Browse files Browse the repository at this point in the history
When a React PR is opened CI will report large size changes. But for critical packages like react-dom it reports always. In React 19 we moved the build for react-dom the client reconciler from react-dom to react-dom/client

This change adds react-dom-client artifacts for stable and oss channels since that is originally what was being tracked. But since react-dom/client always imports react-dom I left the original react-dom packages as critical as well. They are small but it woudl be good to keep an eye on them
  • Loading branch information
gnoff committed May 2, 2024
1 parent 4508873 commit c4cb747
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions dangerfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,9 @@ const CRITICAL_ARTIFACT_PATHS = new Set([
// We always report changes to these bundles, even if the change is
// insignificant or non-existent.
'oss-stable/react-dom/cjs/react-dom.production.js',
'oss-stable/react-dom/cjs/react-dom-client.production.js',
'oss-experimental/react-dom/cjs/react-dom.production.js',
'oss-experimental/react-dom/cjs/react-dom-client.production.js',
'facebook-www/ReactDOM-prod.classic.js',
'facebook-www/ReactDOM-prod.modern.js',
]);
Expand Down

0 comments on commit c4cb747

Please sign in to comment.