Skip to content

Conversation

@vhscom
Copy link
Contributor

@vhscom vhscom commented Apr 25, 2024

Adds support for TSX parsing and documents type used given ESLint now has two config types. Information about the new FlatConfig from Nicholas Zakas can be found here: https://eslint.org/blog/2022/08/new-config-system-part-2/

Ref: https://typescript-eslint.io/packages/typescript-eslint/ on the set-up.

eslint-visitor-keys "^3.3.0"

"@eslint-community/regexpp@^4.6.1":
"@eslint-community/regexpp@^4.10.0", "@eslint-community/regexpp@^4.6.1":
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we using two versions of this same package?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're not, but one of our transient dependencies may be.

@vhscom vhscom force-pushed the fix/eslint-errors-typescript branch 3 times, most recently from 4387b6d to a713cb2 Compare April 25, 2024 21:47
@vhscom vhscom marked this pull request as ready for review April 25, 2024 21:49
@vhscom vhscom force-pushed the fix/eslint-errors-typescript branch from 4ad0096 to 3db9095 Compare April 25, 2024 22:07
@ZacharyKlein ZacharyKlein self-requested a review April 26, 2024 15:52
ZacharyKlein
ZacharyKlein previously approved these changes Apr 26, 2024
mvolkmann
mvolkmann previously approved these changes Apr 26, 2024
jackkeller
jackkeller previously approved these changes Apr 26, 2024
Copy link
Collaborator

@jackkeller jackkeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@vhscom vhscom merged commit c55a80a into develop Apr 26, 2024
@vhscom vhscom deleted the fix/eslint-errors-typescript branch April 26, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants