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

Fix pre-commit template #80

Merged
merged 1 commit into from
Dec 21, 2016
Merged

Fix pre-commit template #80

merged 1 commit into from
Dec 21, 2016

Conversation

bbondy
Copy link
Contributor

@bbondy bbondy commented Aug 29, 2016

Without this, it actually looks for a file with quotes around it on macOS and probably other platforms too.

The path is already escaped because it is part of an args array.

Without this, it actually looks for a file with quotes around it on macOS and probably other platforms too.

The path is already escaped because it is part of an args array.
@coveralls
Copy link

coveralls commented Aug 29, 2016

Coverage Status

Coverage remained the same at 95.413% when pulling 67aef1d on brave:master into 7154dee on observing:master.

@3rd-Eden
Copy link
Member

Thanks for your contribution! Will be in the next patch release

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.

None yet

3 participants