Skip to content

Align pre-commit hooks to use passed filenames and remove internal git diff logic #3195

@Standing-Man

Description

@Standing-Man

Bug description

Some pre-commit hooks (trailing-whitespace.sh, trailing-newline.sh, taplo.sh, binary-artifacts.sh) currently determine the staged file scope internally using git diff --cached, while also setting pass_filenames: false.

This differs from other hooks (e.g., markdownlint, shellcheck) that rely on filenames passed directly by pre-commit. Aligning these hooks to the same pattern would simplify the implementation and improve consistency.

Deployment

None

Versions

No response

Hardware / environment

No response

Sample code

No response

Logs

No response

Iggy server config

No response

Reproduction

No response

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions