typelevel / typelevel/typelevel-scalafix
TypelevelAs should add `cats.syntax` import
Open
Nobody has claimed this yet.
- Dominant language
- Scala
- Stars
- 27
- Forks
- 8
- PR merge metrics
- No merged PRs in 30d
Description
Version 0.1.5
import cats.effect.Resource
val foo: Resource[A, B] = ???
val x = foo.map(_ => ())
The rule TypelevelAs.as will complain and suggest to rewrite the map line to .void.
Resource does not have a void method :)
The check should probably be limited to types that provide the necessary functions.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the TypelevelAs.as rule and reproduce the Resource example from the issue. Check why the rule suggests .void without the required cats.syntax support or on a type that lacks void; done means the suggestion is valid for supported types and no longer misleads for Resource.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100