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
Re-enable new ambiguity warning under 2.13 #10462
Conversation
Include a quick fix.
val cat = if (currentRun.isScala3) WarningCategory.Scala3Migration else WarningCategory.Other | ||
val fix = List(CodeAction("ambiguous reference", Some(w.msg), List(TextEdit(tree.pos.focusStart, w.fix)))) | ||
runReporting.warning(tree.pos, w.msg, cat, context.owner, fix) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the way to motivate something.foreach { w => block }
syntax is to say, the lambda is a block, much like "anonymous pattern-matching function" is a block (of cases). For this syntax, it introduces a parameter. How slick! Or maybe, "it's like { case w => block }
but you can omit the case
. Hey!
@lrytz we don't need to release-note this... or do we? |
I was hoping to read an explanation. Oh, is this to warn without |
The new warning has a very detailed message, so it probably doesn't need to be mentioned in the release notes. |
It offers a quickfix action for certain errors, includes a number of bug fixes and it introduces a new warning by default (scala/scala#10462). In addition to the scala version bump, we also fix the new compiler warnings and bump the scalafmt version (the previous version failed with the new scala version). Release notes: https://github.com/scala/scala/releases/tag/v2.13.12 Reviewers: Divij Vaidya <diviv@amazon.com>, Satish Duggana <satishd@apache.org>
It offers a quickfix action for certain errors, includes a number of bug fixes and it introduces a new warning by default (scala/scala#10462). In addition to the scala version bump, we also fix the new compiler warnings and bump the scalafmt version (the previous version failed with the new scala version). Release notes: https://github.com/scala/scala/releases/tag/v2.13.12 Reviewers: Divij Vaidya <diviv@amazon.com>, Satish Duggana <satishd@apache.org>
It offers a quickfix action for certain errors, includes a number of bug fixes and it introduces a new warning by default (scala/scala#10462). In addition to the scala version bump, we also fix the new compiler warnings and bump the scalafmt version (the previous version failed with the new scala version). Release notes: https://github.com/scala/scala/releases/tag/v2.13.12 Reviewers: Divij Vaidya <diviv@amazon.com>, Satish Duggana <satishd@apache.org>
It offers a quickfix action for certain errors, includes a number of bug fixes and it introduces a new warning by default (scala/scala#10462). In addition to the scala version bump, we also fix the new compiler warnings and bump the scalafmt version (the previous version failed with the new scala version). Release notes: https://github.com/scala/scala/releases/tag/v2.13.12 Reviewers: Divij Vaidya <diviv@amazon.com>, Satish Duggana <satishd@apache.org>
It offers a quickfix action for certain errors, includes a number of bug fixes and it introduces a new warning by default (scala/scala#10462). In addition to the scala version bump, we also fix the new compiler warnings and bump the scalafmt version (the previous version failed with the new scala version). Release notes: https://github.com/scala/scala/releases/tag/v2.13.12 Reviewers: Divij Vaidya <diviv@amazon.com>, Satish Duggana <satishd@apache.org>
Include a quick fix.
follow-up for #10339