Skip to content

ci: restrict package release workflow to cherry-studio's main branch#13797

Merged
GeorgeDong32 merged 1 commit intoCherryHQ:mainfrom
kovsu:ci-fix
Mar 26, 2026
Merged

ci: restrict package release workflow to cherry-studio's main branch#13797
GeorgeDong32 merged 1 commit intoCherryHQ:mainfrom
kovsu:ci-fix

Conversation

@kovsu
Copy link
Copy Markdown
Collaborator

@kovsu kovsu commented Mar 26, 2026

This CI had run in my fork, but it should only run in cherry studio's main branch.

CleanShot 2026-03-26 at 09 11 59@2x

Copy link
Copy Markdown
Collaborator

@GeorgeDong32 GeorgeDong32 left a comment

Choose a reason for hiding this comment

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

✅ 审查通过

这是一个正确且重要的安全修复,防止 release-packages workflow 在 fork 仓库中意外运行。

审查结果

  • ✅ 安全性:阻止 fork 中意外发布 npm 包
  • ✅ 一致性:与 auto-i18n.ymlgithub-issue-tracker.yml 等现有模式一致
  • ✅ 最小改动:仅添加必要的条件检查

推荐合并。

@GeorgeDong32 GeorgeDong32 merged commit 2919d09 into CherryHQ:main Mar 26, 2026
10 checks passed
@kovsu kovsu deleted the ci-fix branch March 26, 2026 06:18
MyPrototypeWhat pushed a commit that referenced this pull request Mar 30, 2026
…13797)

This CI had run in my fork, but it should only run in cherry studio's
main branch.

<img width="1194" height="102" alt="CleanShot 2026-03-26 at 09 11 59@2x"
src="https://github.com/user-attachments/assets/93cc496e-601e-40a4-a5d9-75cd056f7a66"
/>
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