-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add page about Scala 2 -Xsource:3
to migration guide
#2966
Conversation
-Xsoruce:3
to migration guide-Xsource:3
to migration guide
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.
I notice the 2.13.13 mention here, should I wait until that is released officially?
Yeah I'll do the final merge here. |
The [`@nowarn` annotation](https://www.scala-lang.org/api/current/scala/annotation/nowarn.html) can be used to suppress individual warnings, which also works with fatal warnings enabled. | ||
|
||
The Scala 2.13 compiler implements quick fixes for many Scala 3 migration warnings. | ||
Quick fixes are displayed in Metals-based IDEs (not yet in IntelliJ), and they can be applied directly to the source code using the `-quickfix` flag, for example `-quickfix:cat=scala3-migration`. |
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.
Not yet in IntelliJ? Really? Is there a ticket somewhere tracking that?
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.
Not sure, if we find one we should add it to scala/scala3#14904
new page is online: https://docs.scala-lang.org/scala3/guides/migration/tooling-scala2-xsource3.html I've linked to it from the 2.13.13 release notes and posted or commented about it on both our forums and on Reddit |
I wonder if we should advertise this right now, given scala/bug#12961. Moving to I'll update the page. In any case, the page is entirely missing the fact that case companions are affected by |
👍 I figured it was okay to publish and link to, because in the short run 99% of page visitors will have come there from the 2.13.13 release notes, which already highlight 12961. But yes, of course it would be best if we update it reasonably promptly. |
No description provided.