microsoft / rushstack

Monorepo for tools developed by the Rush Stack community
https://rushstack.io/
Other
5.93k stars 596 forks source link

[eslint-config] Ability to use in pure JavaScript projects #4989

Open rakleed opened 2 days ago

rakleed commented 2 days ago

Summary

I would like to use the config in a project without TypeScript, as it has useful rules regardless of the presence/absence of TypeScript.

Details

By default you only look at ts files and tsconfig.json, although js projects use jsconfig.json: https://github.com/microsoft/rushstack/blob/871e7000a6036393f6e005e00db63ab3ef59be15/eslint/eslint-config/profile/_common.js#L213-L218

It would be convenient if you made a separate config for JS projects, where these features would be taken into account.

Standard questions

Question Answer
@rushstack/eslint-config version? 4.0.2
Operating system? N/A
Would you consider contributing a PR? No
TypeScript compiler version? N/A
Node.js version (node -v)? N/A
iclanton commented 1 day ago

We'd probably take this change, but all of our projects are in TypeScript, and TypeScript is the Rushstack recommended language, so we probably wouldn't put this change together ourselves. If someone else wants to take this on, go for it!