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

change from babel-eslint to @babel/eslint-parser #146

Closed
AndreasChristianson opened this issue Jan 26, 2021 · 1 comment
Closed

change from babel-eslint to @babel/eslint-parser #146

AndreasChristianson opened this issue Jan 26, 2021 · 1 comment
Milestone

Comments

@AndreasChristianson
Copy link

AndreasChristianson commented Jan 26, 2021

problem

I found that I was affected by babel/babel#10904

yarn run v1.22.10
$ eslint .

Oops! Something went wrong! :(

ESLint: 7.18.0

TypeError: Cannot read property 'value' of null

I can provide more context or a mcve if needed.

workaround

I worked around this problem by overriding the parser in the .eslintrc.js that consumes this config:

module.exports = {
    extends: [
        'get-off-my-lawn'
    ],
    parser: '@babel/eslint-parser'
}

potential solution

Could we switch to @babel/eslint-parser as our parser here?

I'm willing to submit a pr for this solution.

@manovotny
Copy link
Owner

manovotny commented Feb 15, 2021

Attempting to fix in #133.

@manovotny manovotny added this to the v6 milestone Feb 15, 2021
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

No branches or pull requests

2 participants