Repository navigation
Add eslintConfig to PackageJson type #371
Description
ESLint allows their configuration to be stored within the package.json. Seeing as TypeScript and Yarn configs are added, I think ESLint deserves a spot there as well.
Activity
sindresorhus commentedon Mar 6, 2022
We cannot possible support every config that can be put into package.json. We support TypeScript as it's only 3 fields and very popular. So not sure about this one.
sindresorhus commentedon Mar 6, 2022
Do you actually need this or are did you just notice it was missing?
voxpelli commentedon Mar 6, 2022
I agree with @sindresorhus, at most it would be as part of something like
PackageJsonExtras, but also nothing stopping eg ESLint themselves to publish a type for their property, which would be even better than us maintaining it here 👍Have you tried asking the ESLint project? If they reject the idea, then we can revisit it here?
pting-me commentedon Mar 6, 2022
Yeah I actually do need this for another project. I've done my own implementation for the time being.
There is a definition in
@types/eslintfor their configuration object:Linter.Config. I'm proposing that thePackageJsoninclude a propertyeslintConfigthat points to that type. I just noticed you don't have any dependencies on other@typespackages, so wanted to check in if that was a specific design decision. Otherwise, the change should be fairly simple, and I can raise a PR.@voxpelli makes a good point about having some sort of
PackageJsonExtras. Perhaps we should consider that for other possiblepackage.jsonadditions (including this one). Thoughts?voxpelli commentedon Mar 8, 2022
Added comment to #372, lets keep the discussion there then