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

WIP: Support Scala 2.13.0-RC2 #1479

Merged
merged 8 commits into from
May 31, 2019

Conversation

rossabaker
Copy link
Member

@rossabaker rossabaker commented May 26, 2019

No description provided.

"org.typelevel" %%% "cats-laws" % "2.0.0-M2" % "test",
"org.typelevel" %%% "cats-effect" % "2.0.0-M2",
"org.typelevel" %%% "cats-effect-laws" % "2.0.0-M2" % "test",
"org.scalacheck" %%% "scalacheck" % "1.14.0" % "test",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we're on cats-laws-2.x and cats-effect-laws-2.x, we should be on scalacheck-1.14.0.

build.sbt Outdated
@@ -73,7 +66,7 @@ lazy val commonSettings = Seq(
implicit val contextShiftIO: ContextShift[IO] = IO.contextShift(global)
implicit val timerIO: Timer[IO] = IO.timer(global)
""",
doctestTestFramework := DoctestTestFramework.ScalaTest,
doctestTestFramework := DoctestTestFramework.ScalaCheck,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rossabaker
Copy link
Member Author

Blocked on scalatest/scalatestplus-testng#3. We should consider porting those tests to reduce test dependency liability.

@rossabaker
Copy link
Member Author

Nope, easier said than done: it (appropriately) uses the reactive-streams-tck, which uses testng.

@rossabaker
Copy link
Member Author

Also blocked on scodec/scodec-bits#118.

@rossabaker rossabaker mentioned this pull request May 26, 2019
11 tasks
@mpilquist
Copy link
Member

I was hoping to get #1478 in first given the ScalaTest changes. WDYT?

@rossabaker
Copy link
Member Author

That looks reasonable, and this is blocked on testng, so 👍*2.

@mpilquist mpilquist marked this pull request as ready for review May 31, 2019 13:47
@mpilquist
Copy link
Member

@rossabaker I updated this and fixed a few test issues introduced by the new test suite (unrelated to this PR).

@rossabaker
Copy link
Member Author

What is the version strategy going to be for this? fs2 doesn't publish laws (I don't think?), so it doesn't have to be major. But cats-effect-2.x will not be binary compatible in 2.11, and if the fromFuture ticket goes through, anywhere. Does this mean fs2-2.0?

@mpilquist
Copy link
Member

Argh, good catch. Maybe 1.1?

@mpilquist
Copy link
Member

I'm going to merge this to series/1.0 at least and then figure out where to go from there.

@mpilquist mpilquist merged commit 2a56a37 into typelevel:series/1.0 May 31, 2019
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

Successfully merging this pull request may close these issues.

2 participants