use env port

[lizzzp1]

#1126
Replaces command.port with process.env.PORT
Adds process.env.PORT as additional default

I think it's ok to just replace? 😅

💻 Examples

🚨 Test instructions

✔️ PR Todo

• Added/updated unit tests for this change
• Filled out test instructions (In case there aren't any unit tests)
• Included links to related issues/PRs

Don't think test are applicable here.

[mischnic]

I think it's ok to just replace? 😅

I think it should rather be command.port || process.env.PORT || 1234, that way the cli flag will take precedence over the env variable (if used)

[lizzzp1]

@mischnic thank you for that feedback! -- Tests are failing. 🤔 Could it be a yarn/node version issue?

[mischnic]

Tests are failing. 🤔 Could it be a yarn/node version issue?

These tests have been failing for a while, they are not related to your changes (I've tried to fix them in #2532 but Windows isn't working yet)

[devongovett]

Thank you! 🙏