-
Notifications
You must be signed in to change notification settings - Fork 196
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: 小计/总计配置重构 #2375
refactor: 小计/总计配置重构 #2375
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Size Change: +24 B (0%) Total Size: 320 kB
ℹ️ View Unchanged
|
你好, @lijinke666 CI 执行失败, 请点击 [Details] 按钮查看, 并根据日志修复。 Hello, @lijinke666 CI run failed, please click the [Details] button for detailed log information and fix it. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #2375 +/- ##
==========================================
+ Coverage 75.77% 76.82% +1.05%
==========================================
Files 257 268 +11
Lines 11994 12188 +194
Branches 2464 2482 +18
==========================================
+ Hits 9088 9363 +275
+ Misses 1398 1286 -112
- Partials 1508 1539 +31
☔ View full report in Codecov by Sentry. |
🎉 This PR is included in version @antv/s2-v2.0.0-next.9 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version @antv/s2-react-v2.0.0-next.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
👀 PR includes
🎨 Enhance
📝 Description
语义保持统一, 去掉一些含义模糊不清的命名
totals
表示总计, 有些又用grandTotials
sub
表示小计, 有些又用subTotals
TODO: 更好的方式
🔗 Related issue link
🔍 Self-Check before the merge