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

Fix #1180 - ExplicitResultTypes for implicit members of local traits #1181

Merged
merged 4 commits into from
Jul 2, 2020

Conversation

giabao
Copy link
Contributor

@giabao giabao commented Jun 27, 2020

Fix #1180
Also fix: ExplicitResultTypes.skipLocalImplicits = false has no effect.

giabao added 3 commits June 27, 2020 23:01
We have already used `mods.exists(_.is[Mod.Final])` to check `defn` has `final` Mod.
So we can also use that logic to check `implicit` Mod.

I found the old code is unnecessarily complex and IntelliJ (with latest Scala plugin - version 2020.1.1204), like me :D, incorrectly mark the `isRuleCandidate` calls red because it don't understand the implicit `Extract[D, Mod]` param of `isRuleCandidate`.
giabao added a commit to giabao/scalafix that referenced this pull request Jun 27, 2020
@mlachkar mlachkar merged commit 307c109 into scalacenter:master Jul 2, 2020
@olafurpg olafurpg mentioned this pull request Jul 10, 2020
bjaglin pushed a commit to bjaglin/scalafix that referenced this pull request Jul 10, 2020
This reverts commit 307c109, reversing
changes made to f7adc0e.
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

Successfully merging this pull request may close these issues.

ExplicitResultTypes don't rewrite implicit members of local traits
3 participants