Skip to content

Lint cats instance imports #7

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
armanbilge opened this issue May 5, 2022 · 0 comments · May be fixed by #31
Open

Lint cats instance imports #7

armanbilge opened this issue May 5, 2022 · 0 comments · May be fixed by #31
Labels
enhancement New feature or request

Comments

@armanbilge
Copy link
Member

There's already a rule for this in cats itself, intended to help with the 2.2.0 upgrade I believe.
https://github.com/typelevel/cats/blob/3314acd0e921140d939aafffd745a98d55556814/scalafix/rules/src/main/scala/fix/Cats_v2_2_0.scala

But since these instances are not deprecated (I assume that there are Reasons), it is easy to use them in new code by mistake (I just cleaned up a use in one of my projects, which is why I thought we might add this).
@armanbilge armanbilge added the enhancement New feature or request label May 5, 2022
@armanbilge armanbilge linked a pull request Jun 20, 2022 that will close this issue
Draft
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Cats RemoveInstancesImport rule armanbilge/typelevel-scalafix
1 participant
@armanbilge