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

Disable lint-byname-implicit because of Scala bug #86

Open
1 task done
mdedetrich opened this issue Jan 20, 2022 · 0 comments
Open
1 task done

Disable lint-byname-implicit because of Scala bug #86

mdedetrich opened this issue Jan 20, 2022 · 0 comments

Comments

@mdedetrich
Copy link
Contributor

What is currently missing?

Currently the Scala compiler is generating warnings with Block result was adapted via implicit conversion (method apply) taking a by-name parameter error but it turns out that this is a false positive because the error is being thrown in library code that uses macros and not actual user written code, see scala/bug#12072

How could this be improved?

Rather than polluting the codebase with @nowarn("cat=lint-byname-implicit") everywhere we should just disable the -Xlint:byname-implicit check until it gets resolved in upstream Scala.

Is this a feature you would work on yourself?

  • I plan to open a pull request for this feature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant