Skip to content

/ pre-commit

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

Jump to bottom

don't use system for ruby/node if it is a shim exe #1668

Merged
merged 1 commit into from Oct 29, 2020
Merged

don't use system for ruby/node if it is a shim exe #1668

merged 1 commit into from Oct 29, 2020

Conversation

@asottile
Copy link
Member

@asottile asottile commented Oct 28, 2020

Resolves #1658
@asottile asottile force-pushed the avoid_shims branch from 8c56ef9 to f3f9fbb Oct 28, 2020
@asottile
don't use system for ruby/node if it is a shim exe
Loading status checks…
7f9f66e
@asottile asottile force-pushed the avoid_shims branch from f3f9fbb to 7f9f66e Oct 28, 2020
@asottile asottile merged commit 763dbc0 into master Oct 29, 2020
2 checks passed
2 checks passed
pre-commit.ci - pr checks completed successfully
Details
pre-commit.pre-commit #20201028.11 succeeded
Details
@asottile asottile deleted the avoid_shims branch Oct 29, 2020
@asottile asottile mentioned this pull request Oct 29, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

rbenv FileNotFoundError when installing markdownlint hook
1 participant
@asottile
You can’t perform that action at this time.