-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Introduction of mutable latest tag broke pre-commit auto-updates #170
Comments
It is common practice for GitHub Actions to publish (mutable) Major and Minor version tags in addition to the static full version tags. I was just following the same practice. Apparently pre-commit cannot handle that and incorrectly offers I'm ok with removing the |
The |
@Jason3S Thanks. To be honest, I had the same kind of problem with ansible-lint action and what I decided to do was to keep the github-action in a separated repository than the tool itself, so I can use |
I have published |
Introduction of mutable latest tag broke pre-commit auto-updates because pre-commit will propose updating the version to
latest
and will no longer update it after that, even if the tag is moved to point to another commit.That behavior is by design as pre-commit assumes that version tags are immutable or otherwise it will never be able to cache them.
There is a thread on https://twitter.com/sbarnea/status/1546778466821054465 related to this issue where the pre-commit author replied. Apparently there is something you can do to address the issue without having to remove the use of
latest
tag but that is really clear to me what needs to be done.The text was updated successfully, but these errors were encountered: