Skip to content
This repository has been archived by the owner on Nov 21, 2019. It is now read-only.

Make scripts for the two gulp actions. Update readme to point to them. #730

Merged
merged 1 commit into from
Jul 13, 2017

Conversation

wbobeirne
Copy link
Contributor

The current instructions to npm install then run gulp aren't quite right. That only works if you have gulp-cli installed globally already. Otherwise, you get -bash: gulp: command not found. You would have to run node_modules/.bin/gulp if you wanted to run it directly from the command line, as your $PATH isn't node_modules aware.

However, package.json scripts are node_modules aware. All this PR does is add a dev and a dist script that runs gulp and gulp clean respectively. I also updated the readme to suggest running those two instead.

@tayvano tayvano merged commit 677ba55 into MyEtherWallet:mercury Jul 13, 2017
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