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

setup.cmd should make pre-commit hooks executable #823

Closed
rjyounes opened this issue Apr 24, 2023 · 0 comments · Fixed by #824
Closed

setup.cmd should make pre-commit hooks executable #823

rjyounes opened this issue Apr 24, 2023 · 0 comments · Fixed by #824
Assignees

Comments

@rjyounes
Copy link
Collaborator

At least on Mac, when setup.cmd runs and copies the pre-commit hook into ./git/hooks, it needs to be made executable - after the copy. If there isn't already an executable file in the hooks directory, it gets copied as non-executable, regardless of the status of the original file in tools.

We should also make the pre-commit in the serializer executable, in case it hasn't come in that way from the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant