Skip to content

ci: repair dead-link check failures - #629

Merged
wangl-cc merged 1 commit into
mainfrom
codex/fix-docs-deadlinks
Oct 23, 2026
Merged

wangl-cc merged 1 commit into
mainfrom
codex/fix-docs-deadlinks

Conversation

@wangl-cc

@wangl-cc wangl-cc commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

动机 / Motivation

修复文档检查中 CHANGELOG 贡献者主页返回 404 的问题,按原 PR #289 当前作者更新署名;fork PR 的 token 只有读权限,跳过其自动评论步骤,避免额外的权限错误。

影响与验证 / Impact and testing

无 CLI 行为变化。cargo +nightly fmt、cargo clippy、actionlint、CHANGELOG 的 markdownlint 和 git diff --check 均通过;新主页返回 HTTP 200。cargo x test 因本机缺少 MaaCore,在测试执行前退出。GitHub 评论步骤尚未在实际 workflow 中验证。

AI 辅助 / AI assistance

使用 Codex 编写并检查上述两处改动。

  • 我已审阅全部改动,能解释每一处修改 / I have reviewed all changes and can explain each of them

Sourcery 摘要

修复文档死链检查,并确保向来自 fork 仓库的贡献者发送拉取请求通知时安全无虞。

错误修复:

  • 修复因 CHANGELOG 中过时的贡献者链接导致的死链工作流失败。
  • 阻止来自 fork 仓库的拉取请求尝试发布未经授权的自动失败评论。

CI:

  • 将死链失败评论限制为仅针对源自本仓库的拉取请求。

文档:

Original summary in English

Summary by Sourcery

Repair documentation dead-link checks and make their pull request notifications safe for forked contributions.

Bug Fixes:

  • Fix dead-link workflow failures caused by outdated CHANGELOG contributor links.
  • Prevent forked pull requests from attempting unauthorized automatic failure comments.

CI:

  • Restrict dead-link failure comments to pull requests originating from the repository.

Documentation:

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗨——我已经审查了你的更改,整体看起来很棒!


Sourcery 对开源项目免费——如果你喜欢我们的审查意见,欢迎分享 ✨
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@wangl-cc wangl-cc changed the title fix(ci): repair dead-link check failures ci: repair dead-link check failures Oct 23, 2026
@wangl-cc
wangl-cc added this pull request to the merge queue Oct 23, 2026
Merged via the queue into main with commit a384523 Oct 23, 2026
20 checks passed
@wangl-cc
wangl-cc deleted the codex/fix-docs-deadlinks branch October 23, 2026 17:05
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.

1 participant