eslint / rewrite

Monorepo for the new version of ESLint
Apache License 2.0
65 stars 4 forks source link

Change Request: eslint/core as types-only package (to start) #62

Closed nzakas closed 4 days ago

nzakas commented 1 week ago

Which packages would you like to change?

What problem do you want to solve?

With language plugins, there a bunch of new type definitions that would be helpful to share from the start (for Language, SourceCode, etc.).

What do you think is the correct solution?

Because eslint/core will be the eventual home of runtime-agnostic ESLint APIs, I think it makes sense to start by creating this package to publish the types that are necessary for language plugins, and then slowly add more APIs and types over time as the rewrite unfolds.

(An alternate approach would be to update @types/eslint with this information, but that would create a dependency on DefinitelyTyped for publishing, which I'm not in favor of. If we publish types in eslint/core, then @types/eslint can always import those to keep things in sync.)

Participation

Additional comments

No response

nzakas commented 6 days ago

I've got one 👍 from @fasttime so I'm going to move forward under the assumption that we'll accept this and can work out the details in a PR.