Skip to content

test(hooks): harden pre-commit optional tool coverage - #11

Merged
taigrr merged 2 commits into
masterfrom
cd/pre-commit-optional-lfs
Oct 19, 2026
Merged

taigrr merged 2 commits into
masterfrom
cd/pre-commit-optional-lfs

Conversation

@taigrr

@taigrr taigrr commented Sep 5, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • skip the pre-commit LFS tracked-file lookup when git-lfs is not installed
  • add a regression test that runs pre-commit with git available but git-lfs absent
  • cover pre-commit optional gitleaks delegation and blocking exit behavior

Tests

  • for test_script in test/*_test.sh; do bash "$test_script" || exit 1; done
  • shellcheck pre-commit commit-msg pre-push post-checkout post-commit post-merge install.sh uninstall.sh test/*_test.sh

@taigrr taigrr changed the title fix(hooks): avoid git lfs lookup when unavailable test(hooks): harden pre-commit optional tool coverage Sep 26, 2026
@taigrr
taigrr merged commit 37e4792 into master Oct 19, 2026
1 check passed
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.

1 participant