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.

Sign up for GitHub

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

gulp and gulp-git are required dependencies for at least v2.6.0 #153

Closed
colinrymer opened this issue Mar 3, 2016 · 3 comments · Fixed by #154
Closed

gulp and gulp-git are required dependencies for at least v2.6.0 #153

colinrymer opened this issue Mar 3, 2016 · 3 comments · Fixed by #154

Comments

@colinrymer
Copy link

Both gulp and gulp-git are required to run cz-cli, but are only listed as devDependencies, not normal dependencies.
@colinrymer colinrymer mentioned this issue Mar 3, 2016
Closed
@marionebl marionebl mentioned this issue Mar 3, 2016
Merged
@marionebl
Copy link
Contributor

Sorry for introducing this with a8c40a9. I was a bit overzealous optimizing and getting rid of the graceful-fs deprecations there.

@colinrymer
Copy link
Author

No worries. I was glad to see stuff get moved to devDeps, and the bug was easily resolved by just manually installing gulp and gulp-git for now. 👍

@jimthedev
Copy link
Member

I'll try to merge Marios PR tonight when at a computer again
On Thu, Mar 3, 2016 at 4:20 PM Mario Nebl notifications@github.com wrote:

Sorry for introducing this with a8c40a9
a8c40a9.
I was a bit overzealous optimizing and getting rid of the graceful-fs
deprecations there.

—
Reply to this email directly or view it on GitHub
#153 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: make gulp depedencies available for prod installs
3 participants
@jimthedev @colinrymer @marionebl