Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

让工作流不会因为没有 Tag 而跳过 #4040

Merged
merged 14 commits into from
Jun 15, 2024
Merged

Conversation

allMagicNB
Copy link
Contributor

@allMagicNB allMagicNB commented Jun 15, 2024

加了个 --always 参数,同时让工作流能被手动触发,因为利于 Fork 调试(反正主 Repo 用不着)

@allMagicNB allMagicNB requested a review from LTCatt June 15, 2024 10:40
@Silverteal
Copy link
Contributor

@3gf8jv4dvSilverteal#10 提到了另一种写法,供参考。

@LTCatt LTCatt merged commit 9b6e92b into Hex-Dragon:main Jun 15, 2024
5 checks passed
@JingHai-Lingyun JingHai-Lingyun added 完成 已被处理,将在下次更新之后生效 · 优化 labels Jun 15, 2024
@WorldHim
Copy link
Collaborator

WorldHim commented Jun 16, 2024

提点建议

  1. 交 pr 时尽量规范化 commit,最大程度避免根本看不懂意图的 commit 名(e.g. 更新 xxx)
  2. 同一意图 commit 强烈建议不要分拆(除非工作量太大需要分步完成)
  3. 如果尚在实验阶段建议先在本地成功之后再 commit,只保留正确修改,避免改来改去

@LTCatt LTCatt added · Bug and removed · 优化 labels Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· Bug 完成 已被处理,将在下次更新之后生效
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants