-
Notifications
You must be signed in to change notification settings - Fork 407
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
support Scala 2.13.0-M5 (and drop M4) #427
Conversation
that's odd, Travis-CI doesn't seem to be firing up. |
I get the following test failures (Travis doesn't run because the branch is not included in the Travis builds):
|
@rickynils Looks like something that |
we also had some serialization trouble in the scala-xml repo. I've made this umbrella ticket so we don't forget to investigate: scala/scala-dev#562 |
@alexarchambault
|
SerializabilitySpecification success if 🤔 |
Doing that in some of projects of mine too… Not sure why this is necessary (default class loader set up in tests, with some kind of filtering, confuses Java serialization?) |
Yup, that helps here, too:
|
- scala: 2.13.0-M4 | ||
env: PLATFORM=js SBT_PARALLEL=true WORKERS=1 DEPLOY=true SCALAJS_VERSION=1.0.0-M3 | ||
- scala: 2.13.0-M5 | ||
env: PLATFORM=js SBT_PARALLEL=true WORKERS=1 DEPLOY=true SCALAJS_VERSION=0.6.25 |
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.
@SethTisue updated 0.6.25 in Travis, but missed the value inplugins.sbt
, as well
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.
Actually, this is an exclude rule, so I'm wrong about the first part, but I think I'm right about the second part. It's important because I don't think Rickard uses Travis to deploy
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.
(see #427 (comment)) (cherry picked from commit a3b07d2)
references #418