- Do you want to request a _feature_ or report a _bug_?
Enhancement?
- What is the current behavior?
People often get confused about port and targetPort options because they sound similar and don't convey enough obvious meaning to new users.
We should rename either targetPort or both of them to something more obviously meaningful.
- If the current behavior is a bug, please provide the steps to reproduce.
- What is the expected behavior?
People should be able to guess right away, which port means what.
- Local Environment Information
โโโโโโโโโโโโโโโโโโโโโ
Environment Info โ
โโโโโโโโโโโโโโโโโโโโโ
System:
OS: macOS 10.15.4
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Binaries:
Node: 10.20.1 - ~/.n/bin/node
Yarn: 1.17.3 - /usr/local/bin/yarn
npm: 6.14.4 - ~/.n/bin/npm
Browsers:
Chrome: 81.0.4044.138
Firefox: 76.0
Safari: 13.1
npmGlobalPackages:
netlify-cli: 2.51.0
netlify-cli/2.51.0 darwin-x64 node-v10.20.1
"target" can be interpreted too many ways, so causes confusion. I suggest the variables be named as follows:
port to be renamed:
netlifydev_will_publish_the_build_on_this_port
and targetPort be renamed:
netlifydev_will_listen_for_your_app_on_this_port
Those names are descriptive, hard to misinterpret, and therefore good. So what if they are sentences if it eliminates confusion and makes the code easier to read.
Thank you for creating this issue. The port stuff has been confusing since my first use of the cli.
port and netlifyDevPort would been enough of a hint for me to avoid the confusion.
portandnetlifyDevPortwould been enough of a hint for me to avoid the confusion.
Sooo... port would be the port of my app? And netlifyDevPort would be the port I visit after running netlify dev command?
port currently means the port of Netlify Dev server. I think changing the meaning of that will lead to things breaking and confusion. We should move away from port and targetPort entirely
Most helpful comment
"target" can be interpreted too many ways, so causes confusion. I suggest the variables be named as follows:
portto be renamed:and
targetPortbe renamed:Those names are descriptive, hard to misinterpret, and therefore good. So what if they are sentences if it eliminates confusion and makes the code easier to read.
Thank you for creating this issue. The port stuff has been confusing since my first use of the cli.