Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Would be nice to bring back a few combined configs #169

Open
mgcrea opened this issue Apr 11, 2023 · 2 comments
Open

Would be nice to bring back a few combined configs #169

mgcrea opened this issue Apr 11, 2023 · 2 comments

Comments

@mgcrea
Copy link
Contributor

mgcrea commented Apr 11, 2023

Being able to expand multiple config is great but for most modern Node.js project we want:

  "extends": [
    "@tsconfig/node-lts/tsconfig.json",
    "@tsconfig/strictest/tsconfig.json",
    "@tsconfig/esm/tsconfig.json"
  ],

Which requires three devDependencies and can be seen as a bit bloated.

    "@tsconfig/esm": "^1.0.2",
    "@tsconfig/node-lts": "^18.12.1",
    "@tsconfig/strictest": "^2.0.0",

It might make sense to ship a combined config (that leverages the new array-support for extends) for the most common ones (like. lts-strictest-esm).

Would be great! Thanks!

@orta
Copy link
Member

orta commented Apr 12, 2023

I think maybe #135 could include all the other dependencies, so you could have the package json with @tsconfig/bases and then still use the same tsconfig references

@mgcrea
Copy link
Contributor Author

mgcrea commented Apr 20, 2023

Only downside I see is that the version would bump for every change in any config. I think, in my case, it would still be more convenient to have this combined config back but it's a minor issue. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants