Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added phoenix task #358

Closed
wants to merge 1 commit into from
Closed

Conversation

nucliweb
Copy link
Contributor

@nucliweb nucliweb commented Aug 4, 2016

Added new task phoenix for remove node_modules and reinstall all dependencies.

"phoenix": "rm -rf node_modules && npm install"

@ghost ghost added the CLA Signed label Aug 4, 2016
@gaearon
Copy link
Contributor

gaearon commented Aug 4, 2016

I don’t think this command is valuable enough to justify adding and documenting it. The suggested implementation also doesn’t work on Windows. Thanks for PR though!

@gaearon gaearon closed this Aug 4, 2016
@nucliweb nucliweb deleted the feature/phoenix branch August 4, 2016 15:13
@nucliweb
Copy link
Contributor Author

nucliweb commented Aug 4, 2016

For add support to Windows, the script needs dependencies... rimraf and npm-run-all

@gaearon
Copy link
Contributor

gaearon commented Aug 4, 2016

Yea, I know, but I just don’t think it’s worth it.

@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants