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

suggest gulp-cli in the getting started #1510

Merged
merged 1 commit into from
Feb 2, 2016
Merged

Conversation

phated
Copy link
Member

@phated phated commented Jan 31, 2016

Closes #1451

@contra How does this look?

@yocontra
Copy link
Member

yocontra commented Feb 1, 2016

@phated is the collision an issue?

@phated
Copy link
Member Author

phated commented Feb 1, 2016

In old versions of npm (ones that ship with many versions of node), if you did:

  1. npm install -g gulp
  2. npm install -g gulp-cli
  3. npm rm -g gulp
    It would remove the gulp bin because it knew that gulp could create it but it didn't check if the bin was owned by gulp. We had this fixed as part of our issues in a 3.x release but I don't know if it ever got backported to 2.x and many people, myself included, still use 2.x for speed, etc.

@yocontra
Copy link
Member

yocontra commented Feb 1, 2016

@phated Mind testing it on 2.x?

@phated
Copy link
Member Author

phated commented Feb 1, 2016

I'd prefer not to, I have gulp-cli linked globally to work on it and the switching between linked and installed is a nightmare due to manpage generation. Even if this was fixed in 2.x, 1.4.x is still out in the wild and shipping with versions of node (0.10.x)

@yocontra
Copy link
Member

yocontra commented Feb 2, 2016

@phated okay.

yocontra pushed a commit that referenced this pull request Feb 2, 2016
suggest gulp-cli in the getting started
@yocontra yocontra merged commit 30c4e53 into master Feb 2, 2016
@phated phated deleted the gulp-cli-suggestion branch February 2, 2016 00:04
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.

2 participants