Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

npm serve broken on windows #16

Closed
nicolasgarnier opened this issue Dec 15, 2017 · 5 comments
Closed

npm serve broken on windows #16

nicolasgarnier opened this issue Dec 15, 2017 · 5 comments
Assignees

Comments

@nicolasgarnier
Copy link
Contributor

From @lxnyce on May 22, 2016 17:7

The following command doesn't work in windows CLI.

"serve": "bower install; firebase serve & babel -w scripts -s --retain-lines -d lib"

See here for some more info : http://stackoverflow.com/questions/30950032/how-can-i-run-multiple-npm-scripts-in-parallel

Copied from original issue: googlearchive/friendlypix#4

@nicolasgarnier
Copy link
Contributor Author

From @tomtom215 on September 10, 2016 3:50

Here is my work around for package.son for those on windows experiencing errors. I left the original ones in for easy comparison. Just run Bower install manually and then win* scripts will work the same as the originals.

{
"scripts": {
"build": "babel scripts -s --minified --compact true -d lib; bower install; echo 'Done. Ready to deploy.'",
"win-build": "babel scripts -s --minified --compact true -d lib",
"serve": "firebase serve & babel -w scripts -s --retain-lines -d lib",
"win-serve":"firebase serve && babel -w scripts -s --retain-lines -d lib"
},

@nicolasgarnier
Copy link
Contributor Author

From @llanzo on January 2, 2017 0:10

Thanks for package.json workaround, for anyone else who ends up here, make sure to double check the scripts path. Following the example provided by @tomtom215 I modified my package.json on windows as follows :
{
"scripts": {
"build": "babel public/scripts -s --minified --compact true -d public/scripts-es5; bower install; echo 'Done. Ready to deploy.'",
"win-build": "babel public/scripts -s --minified --compact true -d lib",
"serve": "bower install; firebase serve & babel -w public/scripts -s --retain-lines -d public/scripts-es5",
"win-serve":"firebase serve && babel -w scripts -s --retain-lines -d lib"
},

@nicolasgarnier
Copy link
Contributor Author

Hey guys. I finally got to fix this. Normally the scripts should be windows compatible now (using concurrently to run tasks in parallel). Feel free to try it out.

@fonix232
Copy link

fonix232 commented Apr 6, 2018

Since it's merged, the readme should be updated too, no?

@nicolasgarnier
Copy link
Contributor Author

👍 Totally. Thanks for the heads up. I forgot about the readme :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants