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

[Package Update] Switch from babel-eslint to @babel/eslint-parser to get latests fixes #10636

Closed
mrnossiom opened this issue Mar 2, 2021 · 4 comments · Fixed by #10761
Closed

Comments

@mrnossiom
Copy link

Hello there,

I ran eslint on my project and got an error, I searched on the internet and found that this was fixed in the latest version of the babel eslint parser but that they moved to another package name.

It went from babel-eslint to @babel/eslint-parser.

Could eslint-config-react-app switch to this new version ?

Here's a link that explain babel's decision: Babel Blog

Thanks for any answers...

MrNossiom

@mrnossiom
Copy link
Author

Maybe I could have an answer...
I can see that I'm not the only one bothered by this.

I can do a PR to upgrade the package, just ask...

@agiveygives
Copy link

In the mean time, babel/babel-eslint#799 (comment) worked for me in fixing the cannot read property 'value' of null issue on builds.

@mrnossiom
Copy link
Author

mrnossiom commented Jun 12, 2021

In the mean time, babel/babel-eslint#799 (comment) worked for me in fixing the cannot read property 'value' of null issue on builds.

Yes, but still, this package is not maintained any more and could break.

It will work as a temporary workaround...

EDIT: the PR #10761 is fixing that and should be merged in the next days or weeks.

@mrmckeb
Copy link
Contributor

mrmckeb commented Jun 12, 2021

I'll close this off as I'm working on this right now and it will be released in 4.1.

Sorry it took us a while to get this out!

@mrmckeb mrmckeb closed this as completed Jun 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants