Skip to content

chore(workflow): add pr-size-label workflow file#6381

Open
3gf8jv4dv wants to merge 6 commits into
HMCL-dev:mainfrom
3gf8jv4dv:chore/workflow-pr-size-label
Open

chore(workflow): add pr-size-label workflow file#6381
3gf8jv4dv wants to merge 6 commits into
HMCL-dev:mainfrom
3gf8jv4dv:chore/workflow-pr-size-label

Conversation

@3gf8jv4dv

@3gf8jv4dv 3gf8jv4dv commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

前段时间我在 #4921 写了篇评论。由于没看到明确的反对、疑问等,我先整理了这个 PR 作为一个尝试。

这个 PR 添加了一个为各 PR 标注代码规模 label 的 workflow。在 https://github.com/UNIkeEN/SJMCL/blob/4b511c8a5adb99de34a123ccf08def036ad29735/.github/workflows/pr-size-label.yml 的基础上改进而来。

在 SJMCL workflow 的基础上,

  • 修改了忽略的目标文件,现在会排除各类非业务文件
  • 添加了一个手动触发机制,为所有无 label PR 自动添加表示 PR 更改规模的 label

合并后,在 Actions 页面点击左侧的「PR Size Label」,然后点击页面中的「Run workflow」按钮,在 main 分支上运行即可为所有无 label 的 PR 标注 label。

如果要采纳这个 PR,需要先在 https://github.com/HMCL-dev/HMCL/labels 页面添加以下 label:

Name Description Color
1+ 0-9 code lines changed. #009900
10+ 10-39 code lines changed. #77bb00
40+ 40-99 code lines changed. #eebb00
100+ 100-499 code lines changed. #ee9900
500+ 500-999 code lines changed. #ee5500
1000+ 1000-1999 code lines changed. #ee0000
2000+ 2000-4999 code lines changed. #ee0000
5000+ 5000+ code lines changed. #ee0000

我在自己的 fork 里测试了下,目前整理了一批 PR 作为展示:https://github.com/3gf8jv4dv/HMCL/pulls

也可以看看 SJMCL 的实际运作效果:https://github.com/UNIkeEN/SJMCL/pulls

@Glavo

Glavo commented Jul 12, 2026

Copy link
Copy Markdown
Member

我感觉还行?就是 size 用在这里感觉有点抽象。

@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

我感觉还行?就是 size 用在这里感觉有点抽象。

这个好说,我想想哪个词合适。

@zkitefly

Copy link
Copy Markdown
Member

description 我觉得简单点就好,例如直接 changes 0-9 lines changes 10-29 lines 这样

@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

description 我觉得简单点就好,例如直接 changes 0-9 lines changes 10-29 lines 这样

我改了下 PR 描述,再看看?

@Wulian233

Copy link
Copy Markdown
Contributor

我个人而言对尺码大小没有明确的概念。我推荐使用i18n CFPA社区的以改动行数作为标签,并移除size/。如
10+
100+
1000+
5000+等

https://github.com/CFPAOrg/Minecraft-Mod-Language-Package/pulls

@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

@Wulian233 改好了。https://github.com/3gf8jv4dv/HMCL/pulls 的 label 也全部更新了。

@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

我计划排除更多的非业务文件,例如各类文档、工作流、配置文件等。

@Wulian233

Copy link
Copy Markdown
Contributor

LGTM

@Glavo

Glavo commented Jul 12, 2026

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 016ec2a80f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/pr-size-label.yml Outdated
@3gf8jv4dv 3gf8jv4dv marked this pull request as draft July 12, 2026 14:35
@3gf8jv4dv 3gf8jv4dv marked this pull request as ready for review July 12, 2026 14:52
@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

更新了 workflow,计算行数时会排除更多非业务文件——各类文档、配置文件、资源文件等。

@3gf8jv4dv

Copy link
Copy Markdown
Contributor Author

感觉差不多了。如果还有疑问,可以继续评论或念焚诀。

此外,致维护者的注意事项

  1. 一定要先在 https://github.com/HMCL-dev/HMCL/labels 添加最上面表格中列出的 label。
  2. 如果要为既有 PR 添加 label,在合并这个 PR、添加 label 后,在 actions 页面中点击左侧的「PR Size Label」,再点击页面中的「Run workflow」按钮,保持默认 main 点击「Run workflow」
  3. 最上面表格中列出的颜色仅供参考。如果维护者有自己的喜好,可以根据实际情况改改。或者如果有更好的建议,可以继续评论。

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.

4 participants