npm start - process.env.PORT support

[dolchi21]

run "PORT=3001 npm start" to start the app at port 3001

[ghost]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

[tyrbo]

Would love it if the host was configurable as well.

[sheepsteak]

If the minimum requirement is Node 4 can't we use template strings for these?

[gaearon]

Can you please describe your use case for customizing the port? Do you want to run two instances of the app? Do you have another tool at this port?

[dolchi21]

Yes. I have an express app at port 3000 behind a nginx proxy. If the react app wants to run on 3000 i would have to modify the nginx conf and the express app.
another:
running the /user and the /admin apps at the same time

[gaearon]

I think we’ll go with something like #101 for now.

We might add support for specifying the port in the future but it seems like #101 would be a good enough solution for most people (and it should cover both your use cases).

What do you think?

[dolchi21]

i think is just a matter of preference. i'm used to ExpressJS where you set env.PORT to whatever you like.
Whatever works is fine with me.

[mxstbr]

Awesome, I'll close this for now then and get #101 merged. Thanks so much for taking the time to contribute @dolchi21!

[gaearon]

Yep. We might revisit this later.