Skip to content

Conversation

@Pierre-Sassoulas
Copy link
Member

Executed locally, the zizmor issue is "action is not pinned to a hash (required by blanket policy)" * 20

@Pierre-Sassoulas Pierre-Sassoulas added the skip news used on prs to opt out of the changelog requirement label Jan 13, 2026
@bluetech
Copy link
Member

Executed locally, the zizmor issue is "action is not pinned to a hash (required by blanket policy)" * 20

I think we should fix it, it's good to pin with hashes, and dependabot supports updating the pin so it shouldn't be a maintenance headache AFAIK. I will look into pushing the pins to this PR in a bit unless you beat me to it :)

@webknjaz
Copy link
Member

dependabot supports updating the pin so it shouldn't be a maintenance headache AFAIK

Make sure to leave # v1.0.0 comments matching the commits in your PR. Dependabot updates such comments with actual tags when in offers bumps. But I think this might get out-of-sync if the version in comment doesn't match the hash at the time of the dependabot run.

@Pierre-Sassoulas Pierre-Sassoulas self-assigned this Jan 13, 2026
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the pre-commit-ci-update-config branch from df76d24 to 0839504 Compare January 13, 2026 15:01
@Pierre-Sassoulas
Copy link
Member

I did that and also fixed the zizmor output in bd1ef86

Copy link
Member

@bluetech bluetech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@Pierre-Sassoulas Pierre-Sassoulas merged commit d794da3 into main Jan 13, 2026
33 checks passed
@Pierre-Sassoulas Pierre-Sassoulas deleted the pre-commit-ci-update-config branch January 13, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news used on prs to opt out of the changelog requirement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants