fix(table-core): support numeric accessor keys#6434
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 (5)
📝 WalkthroughWalkthroughNumeric ChangesNumeric accessor key support
Estimated code review effort: 3 (Moderate) | ~20 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 8502850
☁️ Nx Cloud last updated this comment at |
What changed
accessorKeyvalues when constructing tuple/array columnsDeepValue0and1Why
Numeric tuple accessors are accepted by the public types and documented usage, but column construction treated index
0as absent and called string methods on other numeric indices. This caused valid array-index columns either to lack an ID/accessor or to throw.Validation
@tanstack/table-coretests: 52 files, 1,002 tests passedgit diff --checkpassedCloses #4815
Summary by CodeRabbit
New Features
Bug Fixes
0and empty strings.