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.

Sign up for GitHub

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

Jump to bottom

chore: migrate to @babel/eslint-parser #10761

Merged
merged 10 commits into from
Jul 14, 2021
Merged

chore: migrate to @babel/eslint-parser #10761

merged 10 commits into from
Jul 14, 2021
+17 −13

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Mar 29, 2021

Fixes #10636

I am not familiar with the codebase, let me know I misses something. Thanks!

ianschmitz
ianschmitz reviewed Mar 31, 2021
View reviewed changes
Copy link
Contributor

@ianschmitz ianschmitz left a comment

Choose a reason for hiding this comment

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

Thanks @JLHwung !

packages/eslint-config-react-app/base.js Outdated Show resolved Hide resolved
packages/eslint-config-react-app/package.json Show resolved Hide resolved
@ianschmitz ianschmitz added this to the 4.0.4 milestone Mar 31, 2021
@iansu iansu mentioned this pull request Mar 31, 2021
Closed
21 tasks
@JLHwung JLHwung force-pushed the switch-to-babel-eslint-parser branch from ed6430c to 29ca1b2 Compare March 31, 2021 19:05
@ljosberinn ljosberinn mentioned this pull request Apr 21, 2021
Open
@JLHwung JLHwung force-pushed the switch-to-babel-eslint-parser branch from 5a2da07 to 4afd4a6 Compare April 22, 2021 18:37
@JLHwung
Copy link
Contributor Author

JLHwung commented Apr 22, 2021

The CI reports that react-app can not be loaded by eslint, likely because @babel/eslint-parser is not provided in certain e2e fixture package.json. However I don't know where are these package.json, help will be appreciated.

/cc @ianschmitz

@karlhorky karlhorky mentioned this pull request May 11, 2021
Closed
@karlhorky
Copy link

Is it maybe having to update the yarn.lock.cached file?

https://github.com/JLHwung/create-react-app/blob/f7109764702dcd376c39c25cc9cadb145472f4f4/packages/create-react-app/yarn.lock.cached#L2431-L2441

@karlhorky karlhorky mentioned this pull request May 11, 2021
Closed
@JLHwung JLHwung force-pushed the switch-to-babel-eslint-parser branch from c50ef6b to 3d18c0a Compare May 14, 2021 14:23
@mrmckeb mrmckeb self-assigned this Jun 11, 2021
@mrnossiom mrnossiom mentioned this pull request Jun 12, 2021
Closed
@redact-digital redact-digital mentioned this pull request May 23, 2024
Open
@belanasaikiran belanasaikiran mentioned this pull request May 24, 2024
Open
@redact-digital redact-digital mentioned this pull request May 24, 2024
Open
This was referenced May 24, 2024
Open
Open
@Omrisnyk Omrisnyk mentioned this pull request May 26, 2024
Open
@rawanst rawanst mentioned this pull request May 27, 2024
Open
@Omrisnyk Omrisnyk mentioned this pull request May 28, 2024
Open
@GregBrimble GregBrimble mentioned this pull request May 28, 2024
Open
@Omrisnyk Omrisnyk mentioned this pull request May 29, 2024
Open
This was referenced May 29, 2024
Open
Open
@Jkffff09 Jkffff09 mentioned this pull request May 31, 2024
Open
This was referenced May 31, 2024
Open
Open
Open
@skydrige skydrige mentioned this pull request Jun 3, 2024
Open
This was referenced Jun 3, 2024
Open
Open
@redact-digital redact-digital mentioned this pull request Jun 7, 2024
Open
This was referenced Jun 8, 2024
Open
Open
Open
Open
This was referenced Jun 18, 2024
Open
Open
@DavidChevallier DavidChevallier mentioned this pull request Jun 21, 2024
Merged
@fady73 fady73 mentioned this pull request Jun 28, 2024
Open
@Omrisnyk Omrisnyk mentioned this pull request Jul 2, 2024
Open
@kalembalukasz kalembalukasz mentioned this pull request Jul 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ianschmitz ianschmitz ianschmitz left review comments

@kaicataldo kaicataldo kaicataldo left review comments

@mrmckeb mrmckeb mrmckeb approved these changes

@iansu iansu Awaiting requested review from iansu

@petetnt petetnt Awaiting requested review from petetnt

Assignees

@mrmckeb mrmckeb

Labels
CLA Signed tag: breaking change tag: underlying tools
Projects
None yet
Milestone
5.0
Development

Successfully merging this pull request may close these issues.

[Package Update] Switch from babel-eslint to @babel/eslint-parser to get latests fixes
10 participants
@JLHwung @karlhorky @mrmckeb @iansu @FontEndArt @ryota-murakami @devinrhode2 @ianschmitz @kaicataldo @facebook-github-bot