-
Notifications
You must be signed in to change notification settings - Fork 194
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
test: fix snapshot #590
test: fix snapshot #590
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Walkthrough此次更改主要涉及 Dumi 配置文件 Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #590 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 16 16
Lines 517 517
Branches 140 151 +11
=========================================
Hits 517 517 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (3)
tests/__snapshots__/Select.checkable.spec.tsx.snap
is excluded by!**/*.snap
tests/__snapshots__/Select.multiple.spec.js.snap
is excluded by!**/*.snap
tests/__snapshots__/Select.spec.tsx.snap
is excluded by!**/*.snap
📒 Files selected for processing (1)
- .dumirc.ts (1 hunks)
🧰 Additional context used
base: '/', | ||
publicPath: '/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
需要更新文档中的旧路径引用。
在以下文件中仍存在对 /tree-select/
的引用,需要确认是否需要更新:
tests/Select.multiple.spec.js
src/TreeSelect.tsx
README.md
CHANGELOG.md
🔗 Analysis chain
更改看起来符合预期,但请验证影响。
base
和 publicPath
的更改从 '/tree-select/' 到 '/' 看起来是有意为之的,并且与修复快照的 PR 目标一致。这种更改可能会影响站点的服务方式和资源的访问方式。
请运行以下脚本来验证这些更改的影响:
请确保运行相关的测试套件,特别是快照测试,以确保这些更改解决了问题且没有引入新的问题。
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# 描述:验证 base 和 publicPath 的更改对项目的影响
# 测试:检查是否有其他文件引用了旧的路径
echo "检查对 '/tree-select/' 的引用:"
rg '/tree-select/' --type ts --type js --type jsx --type tsx
# 测试:检查是否有使用 base 或 publicPath 的地方需要更新
echo "检查使用 base 或 publicPath 的地方:"
rg 'base:|publicPath:' --type ts --type js --type jsx --type tsx
# 测试:检查是否有需要更新的文档
echo "检查可能需要更新的文档:"
rg '/tree-select/' --type md
Length of output: 1248
Script:
#!/bin/bash
# 描述:验证 base 和 publicPath 的更改对项目的影响
# 测试:检查是否有其他文件引用了旧的路径
echo "检查对 '/tree-select/' 的引用:"
rg '/tree-select/' --type ts --type js --type tsx
# 测试:检查是否有使用 base 或 publicPath 的地方需要更新
echo "检查使用 base 或 publicPath 的地方:"
rg 'base:|publicPath:' --type ts --type js --type tsx
# 测试:检查是否有需要更新的文档
echo "检查可能需要更新的文档:"
rg '/tree-select/' --type md
Length of output: 1226
Script:
#!/bin/bash
# 描述:验证 base 和 publicPath 的更改对项目的影响
# 测试:检查是否有其他文件引用了旧的路径
echo "检查对 '/tree-select/' 的引用:"
rg '/tree-select/' -g "*.ts" -g "*.js" -g "*.tsx"
# 测试:检查是否有使用 base 或 publicPath 的地方需要更新
echo "检查使用 base 或 publicPath 的地方:"
rg 'base:|publicPath:' -g "*.ts" -g "*.js" -g "*.tsx"
# 测试:检查是否有需要更新的文档
echo "检查可能需要更新的文档:"
rg '/tree-select/' -g "*.md"
Length of output: 1454
Summary by CodeRabbit