Skip to content

ci: fix react-doctor workflow (bump action, drop phantom input)#795

Merged
zombieJ merged 2 commits into
react-component:masterfrom
aojunhao123:fix/react-doctor-remove-github-token
Jul 13, 2026
Merged

ci: fix react-doctor workflow (bump action, drop phantom input)#795
zombieJ merged 2 commits into
react-component:masterfrom
aojunhao123:fix/react-doctor-remove-github-token

Conversation

@aojunhao123

@aojunhao123 aojunhao123 commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

修一下红了的 ci,以及就是我感觉这个 vercel deploy 对 util 这种工具 lib 来说没必要,社区的 pr ci 不能全绿强迫症有点难受

image

Two fixes to .github/workflows/react-doctor.yml.

1. Bump the pinned action v2.2.2v2.2.7 (fixes the failing check)

2. Drop the unsupported github-token input

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 其他
    • 更新 React Doctor 检查工具至 v2.2.7,提升持续集成检查的稳定性。
    • 简化检查步骤配置,保持现有工作流行为不变。

The millionco/react-doctor action declares no github-token input; its
internal github-script steps use github.token automatically. Passing
github-token is an unknown input, which the GitHub Actions language
server flags as an error.
@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

@aojunhao123 is attempting to deploy a commit to the afc163's projects Team on Vercel.

A member of the Team first needs to authorize it.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

React Doctor 工作流更新为固定的 v2.2.7 提交引用,并移除显式传入的 GitHub Token 配置。

Changes

React Doctor 工作流

Layer / File(s) Summary
更新工作流配置
.github/workflows/react-doctor.yml
将 React Doctor 动作固定到指定提交,并删除步骤中的 github-token 输入配置。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

小兔挥耳看代码,
Doctor 引用更新好。
Token 配置轻轻去,
工作流更简洁俏。

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了这次工作流修复:更新 react-doctor 动作并移除多余输入,和变更内容一致。
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.42%. Comparing base (9d7cebc) to head (599848e).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #795   +/-   ##
=======================================
  Coverage   86.42%   86.42%           
=======================================
  Files          39       39           
  Lines        1068     1068           
  Branches      373      373           
=======================================
  Hits          923      923           
  Misses        143      143           
  Partials        2        2           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aojunhao123 aojunhao123 changed the title ci: drop unsupported github-token input from react-doctor ci: fix react-doctor workflow (phantom input + shallow checkout) Jul 12, 2026
@aojunhao123 aojunhao123 force-pushed the fix/react-doctor-remove-github-token branch from 291572b to e3f63f1 Compare July 12, 2026 09:27
@aojunhao123 aojunhao123 changed the title ci: fix react-doctor workflow (phantom input + shallow checkout) ci: drop unsupported github-token input from react-doctor Jul 12, 2026
The action was pinned to v2.2.2 (0b4f4f4) but installs the CLI at
`version: latest`. react-doctor 0.7.5/0.7.6 bumped the JSON report
schemaVersion to 3; v2.2.2's ensure-json-report.mjs only accepts {1,2},
so it rejected every report, overwrote it with a synthetic error, and
exited 1 — failing the job on PRs regardless of scan results. v2.2.7
accepts schema 3.
@aojunhao123 aojunhao123 changed the title ci: drop unsupported github-token input from react-doctor ci: fix react-doctor workflow (bump action, drop phantom input) Jul 12, 2026
@aojunhao123

Copy link
Copy Markdown
Contributor Author

@afc163 大佬帮忙 review 下

@afc163

afc163 commented Jul 13, 2026

Copy link
Copy Markdown
Member

@zombieJ 来看报错日志。

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
util Ready Ready Preview, Comment Jul 13, 2026 6:38am

@zombieJ

zombieJ commented Jul 13, 2026

Copy link
Copy Markdown
Member

其实没啥关系,主要是 Vercel 现在每个 repo 都要 first approve 一下

@zombieJ zombieJ closed this Jul 13, 2026
@zombieJ zombieJ reopened this Jul 13, 2026
@zombieJ zombieJ merged commit eb3c5b9 into react-component:master Jul 13, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants