-
Notifications
You must be signed in to change notification settings - Fork 59
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
Run latest ScalaCheck #1226
Comments
we're using the 1.14.x branch. I don't know if other changes made on master might cause breakage downstream. but we can find out: https://scala-ci.typesafe.com/job/scala-2.13.x-jdk11-integrate-community-build/1911/ (queued) if it turns out there is too much breakage, then in order to evaluate the effect of typelevel/scalacheck#677, you would need to push a branch somewhere that backports the change to 1.14.x |
Ok, good to know. Thanks for kicking off the build. There hasn't been significant changes since the 1.14.3 last December, but it is enough to be worrying. |
https://scala-ci.typesafe.com/job/scala-2.13.x-jdk11-integrate-community-build/1911/ the lagom failure is unrelated (and has since been addressed) breeze:
does this mean anything to you? I reproduced the failures outside dbuild here, if you want to mess with it: https://github.com/SethTisue/advxml/tree/scalacheck-1.15-snapshot |
The breeze compiler error could be the Dotty change, i.e. typelevel/scalacheck#498 or typelevel/scalacheck#667, but I'll need to dig further. The breeze project may just need the Scalafix. The advxml test error is strange. I wonder if ScalaCheck introduced a regression in the generators, or that advxml had a fragile test that got exposed by a ScalaCheck change, or a combination of both. Anyway, neither of these have to do nested properties, so that puts to rest the concern about typelevel/scalacheck#677, IMO. |
Seth,
Would you be able to run the latest ScalaCheck in the build?
I'm wondering if the changes in typelevel/scalacheck#677 cause problems or not.
Looks like ScalaCheck is frozen at a commit going back to May.
Thanks!
The text was updated successfully, but these errors were encountered: