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
Make the method ##
have no parameter list (instead of a single empty one)
#8814
Conversation
This comment has been minimized.
This comment has been minimized.
https://github.com/scala/scala/blob/2.13.x/src/library-aux/scala/Any.scala should be updated together for documentation purpose - scala/bug#5077 |
29ccdbf
to
075545f
Compare
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.
LGTM otherwise! I wonder how breaking this change will be, i.e., how many ##()
are out there. Community build will hopefully give us an idea. Maybe we can make a special case in the type checker to allow it with a deprecation warning.
075545f
to
3ae592d
Compare
CB run: |
3ae592d
to
6cc0f1b
Compare
6cc0f1b
to
fa9ebbc
Compare
The community build seems fine. Since @SethTisue should we include this in 2.13.2? Scalafix-it? |
let's roll @dwijnand would you mind adding a PR description that motivates the change a bit? doesn't need to be extensive, but if we're going to annoy people with a fiddly-seeming change like this, we should say what it's in service of. |
##
have no parameter list (instead of a single empty one)
Done-zo: lightbend-labs/scala-rewrites#16 |
In Dotty, the method
##
has no parameter list (instead of having a single empty one), so this aligns Scala 2.13 to that. It would appear calling.##()
is highly unlikely so this was changed to error immediately (no warnings).