Allow JSON with comments in jsconfig.json
See original GitHub issueIs your proposal related to a problem?
This is a follow-on from #7248. Right now, we don’t support JSONC (JSON with comments) in jsconfig.json
files.
Describe the solution you’d like
After a discussion with @iansu, we see two paths:
- Implement the same solution as we did for TypeScript. This is easy, but would require us making TypeScript a dependency of
react-scripts
. - The above, but instead of installing
typescript
as a dependency ofreact-scripts
, we would install it to the user’s project if they are using ajsconfig.json
file and don’t havetypescript
installed.
Discussion is welcome.
If you’re interested in picking this up, please let us know.
Issue Analytics
- State:
- Created 4 years ago
- Comments:13 (8 by maintainers)
Top Results From Across the Web
jsconfig.json Reference - Visual Studio Code
The jsconfig. json file specifies the root files and the options for the features provided by the JavaScript language service.
Read more >Why normal json files don't allow comments where as tsconfig ...
There is no configuration to allow comments in JSON, because JSON doesn't support comments. JSON-derived formats such as JSON5 do.
Read more >Documentation - What is a tsconfig.json - TypeScript
The tsconfig.json file specifies the root files and the compiler options required to compile the project. JavaScript projects can use a jsconfig.json file ......
Read more >Configuration Files - ESLint - Pluggable JavaScript Linter
JSON - use .eslintrc.json to define the configuration structure. ESLint's JSON files also allow JavaScript-style comments. package.json - create an ...
Read more >Do SFDX projects really need multiple jsconfig.json files?
When I launch VS Code, some extension is adding a jsconfig.json file to every lwc folder. It has the exact same structure each...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@mrmckeb PR #8140 is ready
@mrmckeb we can pin TS version via react-dev-utils in that case, like cross-spawn and chalk.