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
feat: support parsing markdown
via eslint-mdx
natively
#621
Conversation
🦋 Changeset detectedLatest commit: 535c3af 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 |
markdown
via eslint-mdx
natively
We can also support other HTML related ESLint plugins like https://github.com/yeonjuan/html-eslint or https://github.com/un-ts/eslint/tree/main/packages/markup |
@BPScott Any time to review? |
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.
Code looks good to me.
Does this change mean older versions of any packages will stop working or is this a purely forwards-compatible change? I'm wondering if we need to call any "You should update this other package" stuff in the changeset
Added a few polishing thoughts inline as the changeset prettierrc, package patches and dependency changes seem unrelated to the changes to the worker.
@BPScott This PR is totally backwards/forwards compatible and won't break anything. It's a minor enhancement. |
78057a2
to
535c3af
Compare
The following is unnecessary anymore when using with
eslint-mdx
/eslint-plugin-mdx
!