Skip to content
This repository has been archived by the owner on Jan 23, 2021. It is now read-only.

Use npm bin when calling gulp #71

Merged
merged 1 commit into from
Jan 7, 2016
Merged

Use npm bin when calling gulp #71

merged 1 commit into from
Jan 7, 2016

Conversation

jjinux
Copy link
Contributor

@jjinux jjinux commented Dec 17, 2015

Otherwise, you'll get "-bash: gulp: command not found" when gulp is not
installed globally.

Otherwise, you'll get "-bash: gulp: command not found" when gulp is not
installed globally.
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@jjinux
Copy link
Contributor Author

jjinux commented Dec 17, 2015

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@jjinux
Copy link
Contributor Author

jjinux commented Jan 7, 2016

Hey, @jeffposnick, are you going to merge this PR so that I can delete my fork and local repo? ;)

@jeffposnick
Copy link
Contributor

Sorry, @jjinux, I lost track of this over the break. I forget that not everyone already has a global gulp installed, so sure, this LGTM.

jeffposnick added a commit that referenced this pull request Jan 7, 2016
Use `npm bin` when calling gulp
@jeffposnick jeffposnick merged commit d852038 into GoogleChromeLabs:master Jan 7, 2016
@jjinux
Copy link
Contributor Author

jjinux commented Jan 7, 2016

@jeffposnick, thanks!

@jjinux jjinux deleted the jjinux/npm-bin branch January 7, 2016 22:17
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.

3 participants