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

Changed postcss to check for local installation under node_modules/.bin #5092

Closed
wants to merge 1 commit into from

Conversation

ericselin
Copy link
Contributor

@ericselin ericselin commented Aug 17, 2018

Fixes #5091

@CLAassistant
Copy link

CLAassistant commented Aug 17, 2018

CLA assistant check
All committers have signed the CLA.

@RealOrangeOne
Copy link
Contributor

As this value is now generic, it may work better being moved into a more generic location. I'm not familiar with Go's shell execution framework, but if it's possible to mutate the $PATH for all calls, that could be an interesting implementation too.

@bep
Copy link
Member

bep commented Jul 16, 2019

Merged manually, thanks for this and sorry for the delay.

@bep bep closed this Jul 16, 2019
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Local installation of postcss-cli doesn't work on Windows
4 participants