| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Add an alias to the --watch option.
Update cli.md to add -w alias to --watch.
There was a problem hiding this comment.
The tests are failing. Can you fix them?
Sorry, something went wrong.
|
@anonrig still looking for failed checks but I couldn't find the problem yet. Tests are done in Windows but not in Unix like OSs. Which files should I look? |
Sorry, something went wrong.
|
No strong opinion on this. FWIW, as far as I am aware, we haven't added single-letter options in years. Some years ago, someone added a short option for --conditions, but that had to be reverted, and eventually was re-added three years ago using capital -C to avoid using up lowercase letters (see #38755). Other than that, I don't think we've added any short options in at least five years. In my personal opinion, there are some useful short options (-p, -e), but other than that, the better readability of long options almost always outweighs saving a few keystrokes. |
Sorry, something went wrong.
|
You may be right. But for example in tsc I'm used to using the -w flag, like tsc -w -p ., so I tried node -w a few times accidentally. That's why I opened this pull request. |
Sorry, something went wrong.
|
From a developer point of view using -w can be positive, but as @tniessen said, I agree that long description is good for readability, I think we should evaluate this issue according to the need. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
With this pull request, node -w also will be activate watch mode alongside node --watch.