Skip to content

Use the manual hook stage if it exists in the list#597

Closed
ThunderKey wants to merge 1 commit into
cachix:masterfrom
ThunderKey:patch-1
Closed

Use the manual hook stage if it exists in the list#597
ThunderKey wants to merge 1 commit into
cachix:masterfrom
ThunderKey:patch-1

Conversation

@ThunderKey
Copy link
Copy Markdown

Since #415 seems not maintained anymore and it hinders our workflow I've created a simple workaround.

The issue is, that pre-commit without a --hook-stage uses the commit stage.
Now we have ["pre-push" "manual"] as the installStages defined.
Because of that it does not match the current compare and which lead to the the commit stage being used., which contains no tasks.

This change uses the manual stage if it exists anywhere in the installStages list.

Otherwise, installStages = [ "pre-push" "manual" ] will not use a --hook-stage and use the commit stage.
In pre-commit the commit stage is the default hook stage
@myypo
Copy link
Copy Markdown

myypo commented Aug 3, 2025

I am experiencing the same issue and it seems to be fixing the problem.

@sandydoo
Copy link
Copy Markdown
Member

Superseded by #689. Will give you an option to control which stage to run as well.

@sandydoo sandydoo closed this Feb 25, 2026
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.

3 participants