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

feat(eslint-config-react-app): support eslint-plugin-testing-library 4.x #10817

Conversation

MichaelDeBoey
Copy link
Contributor

@MichaelDeBoey MichaelDeBoey commented Apr 12, 2021

eslint-plugin-testing-library v4.0.0 is released 🎉

I updated the ruleset to the recommended react ruleset as can be seen at https://github.com/testing-library/eslint-plugin-testing-library/blob/2a92133b467e749cc4a8b2a95c9fdac76d7f4ca7/lib/index.ts#L72-L79

I also updated all other ESLint related dependencies to latest minor


CC/ @Belco90

@Belco90
Copy link

Belco90 commented Apr 12, 2021

I'd suggest just bumping eslint-plugin-testing-library to v4 without enabling/updating any rule error level. Otherwise, it will be an actual breaking change update. If we just bump the plugin to v4 without changing any of the currently enabled rules, it shouldn't be a major one since those rules would work in the same way, just reporting some extra edge cases not handled properly before.

@MichaelDeBoey
Copy link
Contributor Author

I sure can split this PR into bumping the version and creating another (breaking) PR to update the ruleset, which can then be included in v5.

CC/ @ianschmitz

@MichaelDeBoey MichaelDeBoey changed the title feat(eslint-config-react-app): support eslint-plugin-testing-library 4.x feat(eslint-config-react-app): support eslint-plugin-testing-library 4.x Apr 13, 2021
@aruniverse
Copy link

I'd like to bump this pr. Updating the version of eslint-plugin-testing-library to ^4.9.0 throughout the monorepo should be good enough, the version of @typescript-eslint/* packages will all resolve to the same ver

@Belco90
Copy link

Belco90 commented Jul 15, 2021

I'd like to bump this pr. Updating the version of eslint-plugin-testing-library to ^4.9.0 throughout the monorepo should be good enough, the version of @typescript-eslint/* packages will all resolve to the same ver

Yes, that would be amazing. Also, as mentioned before, I think we should only update eslint-plugin-testing-library without enabling more rules since that would introduce breaking changes. @MichaelDeBoey are you available to do those changes? If not I can generate an alternative PR, but better to reuse yours.

@MichaelDeBoey MichaelDeBoey force-pushed the eslint-plugin-testing-library-v4 branch from 1070777 to b488836 Compare July 15, 2021 12:23
Copy link

@Belco90 Belco90 left a comment

Choose a reason for hiding this comment

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

Thank you @MichaelDeBoey 🎉

Copy link

@aruniverse aruniverse left a comment

Choose a reason for hiding this comment

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

should be fine bumping other deps, but not sure its necessary/mandatory since theyre all using ^ and should pull in the latest anyway

packages/eslint-config-react-app/package.json Show resolved Hide resolved
packages/react-scripts/package.json Show resolved Hide resolved
@karlhorky
Copy link

This can be closed now, I'm assuming?

Because eslint-plugin-testing-library is now at ^5.0.1:

"eslint-plugin-testing-library": "^5.0.1"

@Belco90
Copy link

Belco90 commented Dec 24, 2021

Indeed, it can be closed now.

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

Successfully merging this pull request may close these issues.

5 participants