Skip to content
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

track scalacheck #13

Closed
adriaanm opened this issue Mar 5, 2014 · 11 comments
Closed

track scalacheck #13

adriaanm opened this issue Mar 5, 2014 · 11 comments

Comments

@adriaanm
Copy link
Contributor

adriaanm commented Mar 5, 2014

Currently at "rickynils/scalacheck.git#1.11.1", but master should be fine

@SethTisue SethTisue self-assigned this Jul 30, 2015
@SethTisue
Copy link
Member

moving to 1.12 is tricky because there are source-incompatible changes. gory details in the wip commits at #119

@SethTisue SethTisue removed their assignment Nov 9, 2015
@SethTisue
Copy link
Member

we should look at this again after 2.12.0 is done and we remove the Scalacheck sources from the scala/scala repo

@SethTisue
Copy link
Member

SethTisue commented Oct 5, 2016

it's no longer relevant that the scala/scala repo includes the ScalaCheck sources, because the community build doesn't build Scala anymore, and there is no ScalaCheck in the published Scala jars. so we're free to try again.

there's a good chance it might actually be easy(-ish?) now to move at least to 1.12.x, maybe 1.13.x

@SethTisue SethTisue self-assigned this Oct 5, 2016
@SethTisue
Copy link
Member

this looks very very promising:

I have a WIP branch SethTisue/community-builds#scalacheck-upgrade just going to 1.12.x. a test run very nearly succeeded: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-integrate-community-build/737/consoleFull

only scala-partest failed to build:

[scala-partest] [error] /home/jenkins/workspace/scala-2.12.x-integrate-community-build/target-0.9.5/project-builds/scala-partest-7721fdf5ade65e4fc588bc0ad5df26ec3d9de596/src/main/scala/scala/tools/partest/nest/Runner.scala:586: type mismatch;
[scala-partest] [error]  found   : org.scalacheck.ScalaCheckFramework
[scala-partest] [error]  required: org.scalatools.testing.Framework
[scala-partest] [error]       val f: Framework = new org.scalacheck.ScalaCheckFramework
[scala-partest] [error]                          ^

scala-partest's current ScalaCheck dependency is 1.11.x, so this is probably some trivial 1.11.x vs 1.2.x we could resolve on a branch, and then that branch would get merged onto scala-partest master after 2.12.0 is done

@SethTisue
Copy link
Member

SethTisue commented Oct 7, 2016

yay we're on 1.12.x now.

as for going to 1.13.x, several projects are still on 1.12.x and there are compilation errors because of source-incompatible changes to ScalaCheck

  • scalaz 7.2.x (7.3.x is on 1.13.x, but the change wasn't backported to 7.2.x)
  • spire

three possible solution approaches:

  • use spaces to support both 1.12 and 1.13
  • stick with 1.12 for now and disable tests on any project requiring 1.13
  • move to 1.13 and disable tests on any project requiring 1.12

I think I lean towards the third option? we kind of expect included projects to keep their dependencies generally current, and ScalaCheck 1.13 has been out since February 2016.

@SethTisue
Copy link
Member

one other option would be to compile ScalaCheck 1.13 in its own space (SethTisue@dbfe1d4) but not actually use it anywhere. I'm not sure that really adds enough value to be worth doing — it's not all that likely that there would be some code in 1.13 only that would tickle some Scala bug.

let's just stay on 1.12 for now. parking this ticket.

fyi @rickynils @non @xuwei-k

@SethTisue SethTisue removed their assignment Oct 7, 2016
@non
Copy link

non commented Oct 17, 2016

I'll try to upgrade Spire ASAP.

@xuwei-k
Copy link
Contributor

xuwei-k commented Dec 16, 2016

now spire depends on scalacheck 1.13

typelevel/spire@4268d92f#diff-fdc3abdfd754eeb24090dbd90aeec2ceR7

scalaz 7.2.x (7.3.x is on 1.13.x, but the change wasn't backported to 7.2.x)

scalaz/scalaz#1282 backported

@SethTisue
Copy link
Member

SethTisue commented Dec 16, 2016

scalaz/scalaz#1282 backported

@xuwei-k nice! seriously, that's really helpful.

and, great news about Spire.

I won't have time to return to this ticket until January or maybe February, but it's still very much on my mind. it's the most important pending change to our projects list.

@SethTisue
Copy link
Member

lift-json is the latest project to now require 1.13

@SethTisue
Copy link
Member

tried adding http4s, but it needs cats-laws, which needs 1.13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants