-
Notifications
You must be signed in to change notification settings - Fork 10
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 all releases #9
Comments
Thank you for the suggestion, but I'm not sure if we want to tag a specific version of this repository itself; it is after all just a tiny |
If you don't intend to change anything in the action other than the default mold version https://github.com/rui314/setup-mold/blob/354d1662b2a6f02e5eccc9712f22657621bf645b/action.yml#L6C5-L6C21, then yes - users could just use v1 of the action and explicitly set the mold version instead. But in that case, why do you even create commits such as "Bump mold version to x.y.z"? The users won't use that new default if they still use the old "v1" tag as your readme suggests them to do. |
I usually don't increment the default |
The
README.md
suggests:However the latest and only "v1" tag has not been updated for the recent commits, e.g. https://github.com/rui314/setup-mold/compare/v1..main
It would be best if each "release" gets its own tag. e.g. "v1.0.2" and a major version tag such as "v1" gets updated continously, see https://docs.github.com/en/actions/creating-actions/about-custom-actions#using-release-management-for-actions
This would also help for pinning full-length sha with a version comment for https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
If you'd rather not do tags, I suggest you update your doc to:
See https://docs.github.com/en/actions/creating-actions/about-custom-actions#using-branches-for-release-management
This way users of the action constantly get (possibly breaking) updates.
The text was updated successfully, but these errors were encountered: