search

cds-snc / node-starter-app

Quick start application setup.... because you have to start somewhere.
MIT License
5 stars 3 forks source link

feature/Add windows support #99

Closed sboisvert closed 4 years ago

sboisvert commented 4 years ago

Add info on how to run on windows.

We could also do npm install --save-optional win-node-env if people think it's better to solve it for this package

CalvinRodo commented 4 years ago

In my experience with trying to sell something to devs in Government you need to make it as easy as possible, also the vast majority of government developers do their work on windows machines.

Ideally this should be able to be brought up in windows without any extra work.

timarney commented 4 years ago

@sboisvert @CalvinRodo

Just testing this on a fresh windows box.

Will document further as we go.

Initial issue without that package

NODE_ENV=development nodemon ./bin/www

NODE_ENV is not recognized as an internal or external command, operable program or batch file.

timarney commented 4 years ago

Looking at this further we'll use https://www.npmjs.com/package/cross-env which will work in "all the cases" without the global install.

Outside of that issue "it just works".

Once we get that setup it will truly just work.

sboisvert commented 4 years ago

@timarney Sorry! I didn't see your comments before pushing my changes. I agree that cross env is probably best. Do you want me to kill this PR or change it?

timarney commented 4 years ago

Closing this in favour of: https://github.com/cds-snc/node-starter-app/pull/100