chore: add react-externals to bundle size analysis#2385
Conversation
🦋 Changeset detectedLatest commit: 2319bb1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughAdded an empty changeset file and updated two GitHub Actions workflow matrices to include the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8f1f0a064c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
8f1f0a0 to
2319bb1
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Merging this PR will degrade performance by 21.16%
Performance Changes
Comparing Footnotes
|
Web Explorer#8455 Bundle Size — 727.89KiB (0%).2319bb1(current) vs dae9fed main#8447(baseline) Bundle metrics
Bundle size by type
|
| Current #8455 |
Baseline #8447 |
|
|---|---|---|
384.4KiB |
384.4KiB |
|
341.33KiB |
341.33KiB |
|
2.16KiB |
2.16KiB |
Bundle analysis report Branch chore/react-externals-bundle-siz... Project dashboard
Generated by RelativeCI Documentation Report issue
React MTF Example#11 Bundle Size — 207.47KiB (0%).2319bb1(current) vs dae9fed main#3(baseline) Bundle metrics
|
| Current #11 |
Baseline #3 |
|
|---|---|---|
0B |
0B |
|
0B |
0B |
|
0% |
0% |
|
0 |
0 |
|
3 |
3 |
|
168 |
168 |
|
66 |
66 |
|
46.13% |
46.13% |
|
2 |
2 |
|
0 |
0 |
Bundle size by type no changes
| Current #11 |
Baseline #3 |
|
|---|---|---|
111.23KiB |
111.23KiB |
|
96.24KiB |
96.24KiB |
Bundle analysis report Branch chore/react-externals-bundle-siz... Project dashboard
Generated by RelativeCI Documentation Report issue
React Example#6877 Bundle Size — 237.89KiB (0%).2319bb1(current) vs dae9fed main#6869(baseline) Bundle metrics
|
| Current #6877 |
Baseline #6869 |
|
|---|---|---|
0B |
0B |
|
0B |
0B |
|
0% |
38.5% |
|
0 |
0 |
|
4 |
4 |
|
174 |
174 |
|
69 |
69 |
|
46.44% |
46.44% |
|
2 |
2 |
|
0 |
0 |
Bundle size by type no changes
| Current #6877 |
Baseline #6869 |
|
|---|---|---|
145.76KiB |
145.76KiB |
|
92.13KiB |
92.13KiB |
Bundle analysis report Branch chore/react-externals-bundle-siz... Project dashboard
Generated by RelativeCI Documentation Report issue
Summary by CodeRabbit
Checklist