-
-
Notifications
You must be signed in to change notification settings - Fork 685
feat: add TypeScript IntelliSense support with eslint-typegen #2770
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
Conversation
🦋 Changeset detectedLatest commit: 9e53387 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
896a67b
to
73d68c9
Compare
Thank you for this PR! |
@ota-meshi Implementation:
Should I also add lib/eslint-typegen.d.ts to this PR, or would you prefer CI automation for the distribution strategy? |
7501517
to
557d8e7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, too. Thank you 🙂
Summary
Add TypeScript IntelliSense support via
eslint-typegen
with ESM migration for ESLint v9.0+ compatibility.Changes
eslint-typegen
for TypeScript definitionseslint.config.js
toeslint.config.mjs
(required for eslint-typegen)--legacy-peer-deps
due to ESLint version requirementsBenefits
Screenshot
close #2757