ci: improve bundle size PR table - #8148
Conversation
|
View your CI Pipeline Execution ↗ for commit 69c0103
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview4 package(s) bumped directly, 19 bumped as dependents. 🟩 Patch bumps
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe bundle-size report now tracks gzip, initial gzip, raw, and Brotli values. It preserves these metrics in baselines, reports signed per-metric deltas, skips only fully unchanged scenarios, and documents the expanded output. ChangesBundle size reporting
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This change updates bundle-size PR reporting to show absolute values, deltas, secondary metric changes, and consistently rendered trends; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 2📝 Generate docstrings 💡
🛠️ Fix failing CI checks 💡
🧪 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 |
Bundle Size BenchmarksThis pull request does not affect bundle size in any measured scenario. |
Merging this PR will regress 6 benchmarks
|
Summary
mainin the Current gzip, Initial gzip, Raw, and Brotli cells<pre>for consistent renderingSample output
react-router.full+2 B
-4 B
+12 B
0 B
Testing
CI=1 NX_DAEMON=false pnpm nx run @benchmarks/bundle-size:test:unit --outputStyle=stream --skipRemoteCachegh-pagesbundle-size historygit diff --checkSummary by CodeRabbit
New Features
Documentation
Tests