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(cli): Change yarn install to npm install #53

Merged

Conversation

sawa-zen
Copy link
Contributor

@sawa-zen sawa-zen commented Jan 10, 2022

Resolve #54

Why

The colors.js used in netlify-cli has been broken. This is because the author of colors.js was angry that he wasn't being paid, so he added an infinite loop process and released it. netlify-cli addressed this issue, but it was a fix based on the assumption that npm was being used.

As a workaround please use npm install -g netlify-cli. We publish a shrinkfile file with the CLI that prevents such issues by locking the CLI dependencies during publish time, however yarn doesn't support it, hence it takes the latest (and broken) version of colors.js.
Ref netlify/cli#3981 (comment)

What

Changed yarn install without Docker file to npm install

@hrishikesh-k hrishikesh-k added the type: chore work needed to keep the product and development running smoothly label Sep 18, 2022
@hrishikesh-k hrishikesh-k merged commit 035e9be into netlify:master Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: chore work needed to keep the product and development running smoothly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fail to install netlify cli action
2 participants