chore: standardize repository setup#1491
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
Walkthrough更新了构建与部署配置、示例导入和文档内容，并同步调整了依赖策略与 GitHub Actions 配置。 Changes构建、示例与文档迁移
仓库自动化
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
❌ Deploy failed
📋 Build log (last lines)🤖 Powered by surge-preview |
|||||||||
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request migrates the repository and its examples from the legacy rc-table and related rc- packages to the scoped @rc-component/table, @rc-component/menu, @rc-component/dropdown, and @rc-component/tooltip packages. It also updates configuration files (such as .dumirc.ts, tsconfig.json, package.json, and vercel.json) to support the new build and documentation output directory (docs-dist). The reviewer recommends using __dirname with path.resolve in .dumirc.ts to ensure robust path resolution across different execution environments.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
React Doctor found 26 issues in 22 files · 26 warnings · score 74 / 100 (Needs work) · vs 26 warnings
Reviewed by React Doctor for commit |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1491 +/- ##
==========================================
+ Coverage 96.10% 99.04% +2.93%
==========================================
Files 57 45 -12
Lines 3442 1362 -2080
Branches 639 409 -230
==========================================
- Hits 3308 1349 -1959
+ Misses 129 13 -116
+ Partials 5 0 -5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/codeql.yml:
- Around line 27-43: The CodeQL workflow is pinned to an older fixed commit for
github/codeql-action in Initialize CodeQL, Autobuild, and Perform CodeQL
Analysis; update those action references to the latest stable v4 release and
keep them pinned to the corresponding official commit SHA. Use the
github/codeql-action entries in the workflow as the places to change, and make
sure all three steps stay on the same updated SHA for consistency.
In `@docs/examples/stickyHeaderAndSummary.tsx`:
- Line 3: 示例里的默认导入 Table 上没有挂载 Summary，导致 Table.Summary、Table.Summary.Row 和
Table.Summary.Cell 运行时为 undefined；请在 stickyHeaderAndSummary.tsx 中改用命名导入的
Summary（以及需要的话 Column/ColumnGroup），并把所有 Table.Summary
相关引用替换为该命名成员，或者如果要保留默认导入用法，则在 src/index.ts 中把 Summary 显式挂到默认导出的 Table 对象上。
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 8ffe63fb-b76a-408e-b18c-57dc3b7d2eee
📒 Files selected for processing (63)
.dumirc.ts.github/FUNDING.yml.github/dependabot.yml.github/workflows/codeql.yml.github/workflows/main.yml.github/workflows/react-doctor.yml.github/workflows/surge-preview.yml.gitignoreREADME.mddocs/examples/animation.tsxdocs/examples/aria.tsxdocs/examples/caption.tsxdocs/examples/childrenIndent.tsxdocs/examples/className.tsxdocs/examples/click-summary-row.tsxdocs/examples/colspan-rowspan-legacy.tsxdocs/examples/colspan-rowspan.tsxdocs/examples/column-hidden.tsxdocs/examples/column-resize.tsxdocs/examples/components.tsxdocs/examples/dropdown.tsxdocs/examples/ellipsis-custom-tooltip.tsxdocs/examples/ellipsis.tsxdocs/examples/expandIcon.tsxdocs/examples/expandedRowClassName.tsxdocs/examples/expandedRowRender.tsxdocs/examples/expandedSticky.tsxdocs/examples/fixedColumns-auto-height.tsxdocs/examples/fixedColumns-resize.tsxdocs/examples/fixedColumns.tsxdocs/examples/fixedColumnsAndHeader.tsxdocs/examples/fixedColumnsAndHeaderRtl.tsxdocs/examples/grouping-columns-hidden.tsxdocs/examples/grouping-columns.tsxdocs/examples/hide-header.tsxdocs/examples/hover-perf.tsxdocs/examples/jsx.tsxdocs/examples/key.tsxdocs/examples/measureRowRender.tsxdocs/examples/nested.tsxdocs/examples/no-data.tsxdocs/examples/react-dnd.tsxdocs/examples/row-hoverable.tsxdocs/examples/rowAndCellClick.tsxdocs/examples/scopeCol.tsxdocs/examples/scopeRow.tsxdocs/examples/scrollX.tsxdocs/examples/scrollXY.tsxdocs/examples/scrollY.tsxdocs/examples/shadow.tsxdocs/examples/simple.tsxdocs/examples/stickyHeader.tsxdocs/examples/stickyHeaderAndSummary.tsxdocs/examples/styled-components.tsxdocs/examples/subTable.tsxdocs/examples/title-and-footer.tsxdocs/examples/virtual-list-grid.tsxdocs/examples/virtual-list.tsxdocs/index.mdpackage.jsontsconfig.jsonvercel.jsonvitest.config.mts
|
Deployment failed with the following error: Learn More: https://vercel.com/afc163s-projects?upgradeToPro=build-rate-limit |
|
Deployment failed with the following error: Learn More: https://vercel.com/react-component?upgradeToPro=build-rate-limit |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/surge-preview.yml:
- Around line 27-31: The Surge preview job is referencing secrets.SURGE_TOKEN
directly in step conditions, which can cause workflow validation failure; move
SURGE_TOKEN to the job-level env for this workflow and update the Build preview
and afc163/surge-preview steps to gate on env.SURGE_TOKEN instead. Use the
existing surge-preview job and the two conditional steps as the place to apply
the change.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b8a87c7b-852a-41ea-ba61-e4acf9b2cd87
📒 Files selected for processing (2)
.github/workflows/surge-preview.ymlREADME.md
✅ Files skipped from review due to trivial changes (1)
- README.md
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
1 participant
Summary
rc-*packages to@rc-component/*packages without changing component runtime behavior.Verification
npm install --ignore-scripts --no-audit --loglevel=warnnpm run tscnpm run lint(passes with existing warnings)npm testnpm run buildnpm run compile(passes with existing warnings)npx vercel build --yesRefs ant-design/ant-design#58514
Summary by CodeRabbit
docs-dist（同时更新 Dumi/忽略/部署配置）。