| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Review requested:
|
Sorry, something went wrong.
Signed-off-by: Marco Ippolito <marcoippolito54@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #62992 +/- ##
==========================================
+ Coverage 89.63% 89.66% +0.03%
==========================================
Files 706 707 +1
Lines 219219 219512 +293
Branches 42004 42088 +84
==========================================
+ Hits 196499 196832 +333
+ Misses 14622 14581 -41
- Partials 8098 8099 +1
... and 60 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Sorry, something went wrong.
|
won't this mean that you can't make a config file that supports multiple versions of node and gracefully uses features from the newer ones when available? |
Sorry, something went wrong.
Sorry, something went wrong.
Yes but also means if you mispell a configuration you will know. I prefer correctness over convenience. |
Sorry, something went wrong.
Sorry, something went wrong.
|
It's not just about convenience, though - it's about being able to support multiple node versions at one time, which also makes upgrading easier (and not supporting that makes upgrading harder). I very much prioritize correctness, but having a closed config every time is highly likely to hold back the ecosystem. |
Sorry, something went wrong.
|
The support for multiple versions of node in the same configuration was never planned and should be discouraged. The $schema needs to match with version being used. I think a possible solution to this problem would be support an array of configurations and node can pick the right one based on version. This plus the ability to extend (like tsconfig) |
Sorry, something went wrong.
Sorry, something went wrong.
Sorry, something went wrong.
Signed-off-by: Marco Ippolito <marcoippolito54@gmail.com> PR-URL: #62992 Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Signed-off-by: Marco Ippolito <marcoippolito54@gmail.com> PR-URL: #62992 Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
| Back | FazBrowse Home | New Git URL |
The documentation is correct but the implementation was not