Skip to content
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

feat: implitic skip on missing files for pre-commit and pre-push hooks #448

Conversation

mrexox
Copy link
Member

@mrexox mrexox commented Mar 15, 2023

Closes #446 #441 #228

⚡ Summary

  • Skip pre-commit command if there's no staged files (with filters applied)
  • Skip pre-push command if there's no push files (with filters applied)
  • Add tests
  • Update docs

Sorry, something went wrong.

@mrexox mrexox changed the title Implitic skip on missing files for pre commit and pre push hooks Implitic skip on missing files for pre-commit and pre-push hooks Mar 15, 2023
@mrexox mrexox force-pushed the feat/implitic-skip-on-missing-files-for-pre-commit-and-pre-push-hooks branch from b889573 to d6402c8 Compare March 15, 2023 16:45

Verified

This commit was signed with the committer’s verified signature. The key has expired.
mrexox Valentin Kiselev
@mrexox mrexox force-pushed the feat/implitic-skip-on-missing-files-for-pre-commit-and-pre-push-hooks branch from d6402c8 to b9b8840 Compare March 15, 2023 17:00

Verified

This commit was signed with the committer’s verified signature. The key has expired.
mrexox Valentin Kiselev
@mrexox mrexox marked this pull request as ready for review March 16, 2023 07:12

Verified

This commit was signed with the committer’s verified signature. The key has expired.
mrexox Valentin Kiselev
@mrexox mrexox changed the title Implitic skip on missing files for pre-commit and pre-push hooks feat: implitic skip on missing files for pre-commit and pre-push hooks Mar 16, 2023
@mrexox mrexox merged commit 8143080 into evilmartians:master Mar 16, 2023
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.

Add a option to skip the command if the file list is empty
1 participant