Skip to content

ci: 增加资源包检查 - #60

Merged
MetaMikuAI merged 2 commits into
mainfrom
ci/resource-pack-check
Nov 1, 2026
Merged

MetaMikuAI merged 2 commits into
mainfrom
ci/resource-pack-check

Conversation

@MetaMikuAI

@MetaMikuAI MetaMikuAI commented Nov 1, 2026 •

Copy link
Copy Markdown
Member

概要

推送和 PR 时自动检查资源包,规则来自编辑器的 scripts/checkResourcePack.ts(MetaMystia/MetaMystia-ResourceEx-Editor#23、#24)。PR 中会自动发一条评论列出错误和警告。

改动

  • 新增 .github/workflows/check-resource.yml:push 到任意分支、PR 和手动触发时,checkout 编辑器 main,用 Node.js 24 运行检查。有错误时检查失败;只有警告时检查通过。PR 中还会上传报告和 PR 编号。
  • 新增 .github/workflows/comment-check-resource.yml:检查结束后读取报告,在 PR 中发布评论,之后的推送更新同一条评论。
    • fork 的 PR 拿不到写权限,所以评论放在单独的 workflow_run workflow 中;它不检出、不运行 PR 中的代码。
    • 报告来自 PR 中运行的 workflow,只有 PR 编号是数字、且该 PR 头提交与本次检查一致时才评论。

合并顺序

  1. 先合并 feat(scripts): add --report option to resource pack check MetaMystia-ResourceEx-Editor#24(--report 选项)。在此之前检查照常运行,但不生成报告,也不会评论。
  2. 再合并本 PR。workflow_run 只按默认分支上的 workflow 运行,评论功能在本 PR 合并后才生效。

验证

  • 本地模拟 CI 目录布局,Node.js 22.22 与 24:0 个错误、11 个警告
  • GitHub Actions 上的检查(push 与 PR)通过,11 条警告以注释形式上传
  • 评论脚本用模拟的 gh 覆盖:新建评论、更新已有评论、PR 已有新提交、缺少报告、PR 编号无效
  • 未验证:合并后 GitHub 上的实际评论

🤖 Generated with Claude Code

推送和 PR 时用编辑器的 scripts/checkResourcePack.ts 检查资源包,有错误时失败,警告只提示。
检查上传报告,comment-check-resource.yml 在检查结束后发布或更新 PR 评论;fork 的 PR 也适用。
@MetaMikuAI
MetaMikuAI merged commit 80b3fbc into main Nov 1, 2026
2 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.

1 participant