thollander / actions-comment-pull-request

GitHub action to comment pull request

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Abstain from renaming branches

ftrippel opened this issue · comments

First: Thank you for all the work you put into this.

One more thing: Please abstain from renaming branches in the future. The github syntax forced us to name a branch when using your github action. So we hardwired "master", and now we have to go through 30 projects to rename it to "main". Actually I would suggest to just stick with conventions or at least not change directions.

Yep. Came here for this. So many projects are broken 😢

But again: Thank you for building this and putting all the effort into it.

Sorry about it, didn't know it had some impact on workflows, I would have put it into a breaking change if so.
You may need to fix the version of the action to a specific one to avoid this.

It may be a little late but you (@thollander) could add a new branch or tag called master to provide backwards compatibility to users that hardwired master into their workflow files. You could either keep it up-to-date forever (in the case of a tag by just --force pushing a newer tag; in the case of a branch by just keeping it pointing to the same commit as main). Alternatively, you could make master a branch with a slightly altered version of the action that always outputs a warning that the user should change to main (or a specific version) - possibly before a specific date so that you can at least remote that master branch at some point.

Additionally, following the official versioning guidelines, I would suggest you switch from recommending the usage of the default branch of your repository to a continuously updated v1 tag, which should always point to the latest v1.x.x version available. This has several advantages, including avoiding problems such as the one described in this issue.

Yes, I gues it's a bit too late to manage it as you suggested (main = master) but that was a good idea.

Totally agree with you, I'll create an issue to manage versionning in a proper way, thanks for taking time for this feedback.