Skip to content

fix pr action error #223

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

Closed
wants to merge 6 commits into from
Closed

fix pr action error #223

wants to merge 6 commits into from

Conversation

Mikachu2333
Copy link
Contributor

问题描述

https://github.com/RubyMetric/chsrc/actions/runs/15900908742
运行错误

解决方案

更改为“非main”分支时提示(万一以后新增大功能开了个branch,有这个action在就没法朝那个新branch pr了)

Copy link

❤️ 感谢你的贡献!你的 PR 当前基于 dev 分支,请修改使用 dev 分支!

@Mikachu2333 Mikachu2333 marked this pull request as draft June 26, 2025 12:07
@ccmywish
Copy link
Contributor

👍 有道理

针对分支这个问题,我有这些看法:

  1. 项目很小,出现功能分支的情况极少
  2. 外部贡献者的贡献大多数是蜻蜓点水,只是一种 patch
  3. 我们的维护很积极,即使有新功能也不至于积累到需要单开一个分支

所以在 dev 上持续开发目前来看是够用的。并且一个 dev 一个 main 很干净,没有维护多个分支的负担。


另外,“推荐dev分支” 由于不是硬性要求,也并不是特别合适。而对于已经熟悉本项目的开发者来说,即使被 actions bot 阻拦,也没有关系,因为我们可以忽视这种阻拦直接审核和合并。


感谢你周到的想法、建议和帮助 ❤️

@ccmywish ccmywish closed this Jun 28, 2025
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.

2 participants