fix(angular-form): prevent full array re-renders in array mode#2171
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughOptimizes TanStack Angular Form change detection to avoid full array re-renders in array mode by tracking array length and splitting reactive selectors; updates docs/examples to use ChangesArray Mode Re-render Optimization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
|
View your CI Pipeline Execution ↗ for commit 425840e
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview5 package(s) bumped directly, 9 bumped as dependents. 🟨 Minor bumps
🟩 Patch bumps
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2171 +/- ##
==========================================
+ Coverage 90.35% 96.51% +6.15%
==========================================
Files 38 7 -31
Lines 1752 86 -1666
Branches 444 3 -441
==========================================
- Hits 1583 83 -1500
+ Misses 149 3 -146
+ Partials 20 0 -20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Mirrors #1963 but for the Angular adapter
Summary by CodeRabbit
Bug Fixes
Documentation