-
Notifications
You must be signed in to change notification settings - Fork 233
chore: Add support for Scala 2.13.15 #3974
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
Conversation
af8f720
to
d32245f
Compare
d32245f
to
e6d0539
Compare
@@ -42,6 +42,6 @@ class Synthetic/*<=example.Synthetic#*/ { | |||
a/*<=local11*/ <- scala.concurrent.Future/*=>scala.concurrent.Future.*/.successful/*=>scala.concurrent.Future.successful().*/(1) | |||
b/*<=local12*/ <- scala.concurrent.Future/*=>scala.concurrent.Future.*/.successful/*=>scala.concurrent.Future.successful().*/(2) | |||
if a/*=>local11*/ </*=>scala.Int#`<`(+3).*/ b/*=>local12*/ | |||
} yield a/*=>local11*/ | |||
} yield a/*=>local13*/ |
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.
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 am assuming you understand the changes, so asked anyway...
if (ScalaVersion.atLeast212_14) body.expected | ||
else | ||
// Predef.type etc. was fixed in 2.12.14 | ||
body.expected.replace("Predef/*=>scala.Predef.*/.type", "Predef.type") |
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.
is this unrelated? cleanup for 2.12?
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.
Yep, that's not necessary anymore and I was already cleaning this up.
@@ -1729,11 +1729,6 @@ Synthetics: | |||
a, | |||
b | |||
) == (1, 2))}).flatMap[Tuple6[Int, Int, Int, Int, Tuple4[Int, Int, Int, Int], Tuple4[Int, Int, Int, Int]]]({(b) => orig(List((a, b))).withFilter({(check$ifrefutable$1) => orig()}).withFilter({(local7) => orig(( | |||
a, |
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.
why did this disappear?
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 would assume that some internal desugarings changed, but I honestly think these synthetics are not being used currently by anyone.
Fixes #3973