Skip to content
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

Incompatible w/ MDX v3? #480

Closed
4 tasks done
jaswrks opened this issue Nov 26, 2023 · 2 comments · Fixed by #486
Closed
4 tasks done

Incompatible w/ MDX v3? #480

jaswrks opened this issue Nov 26, 2023 · 2 comments · Fixed by #486

Comments

@jaswrks
Copy link
Contributor

jaswrks commented Nov 26, 2023

Initial checklist

Affected packages and versions

2.2.0

Link to runnable example

No response

Steps to reproduce

Parsing error: Cannot set properties of undefined (setting 'inTable') eslint

Getting this error after updating to MDX v3.

  • Can someone confirm that eslint-mdx is not yet compatible with MDX v3?
"@mdx-js/mdx": "^3.0.0",


        "eslint": "8.54.0",
        "eslint-plugin-mdx": "2.2.0",

        "remark": "^15.0.1",
        "remark-directive": "^3.0.0",
        "remark-frontmatter": "^5.0.0",
        "remark-gfm": "^4.0.0",
        "remark-lint": "^9.1.2",
        "remark-mdx-frontmatter": "^4.0.0",
        "remark-mermaidjs": "^6.0.0",
        "remark-preset-lint-recommended": "^6.1.3",
        "remark-preset-prettier": "^2.0.1",
        "remark-smartypants": "^2.0.0",
---
Foo: Bar
---

| First Header  | Second Header |
| ------------- | ------------- |
| Content Cell  | Content Cell  |
| Content Cell  | Content Cell  |

Markup on 2023-11-26 at 01:28:43

Expected behavior

ESLint works propertly using MDX v3.

Actual behavior

ESLint error:

Parsing error: Cannot set properties of undefined (setting 'inTable') eslint

Runtime

Node v19

Package manager

npm v9

OS

macOS

Build and bundle tools

Vite

@JounQin
Copy link
Member

JounQin commented Nov 26, 2023

PR welcome, I think that would be very smooth by just upgrading dependencies.

@jaswrks
Copy link
Contributor Author

jaswrks commented Nov 26, 2023

Thanks. PR: #481

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants