Skip to content

Conversation

@felixbr
Copy link
Contributor

@felixbr felixbr commented Jun 17, 2021

Note: This is split out from the big WIP PR #290 and can be done separately as preparation for Scala 3.

Problem

Scala 3 requires Scalacheck 1.15.x but the build currently uses 1.14.x

Solution

Upgrading Scalacheck should be fairly simple as I've yet to see any binary compatibility issues between 1.14.x and 1.15.x.
If we can use the newer version across all Scala versions it will simplify Scala 3 cross-building (otherwise we'd have to use different versions which can be finicky).

Result

The tests use Scalacheck 1.15.4.
We still use scalatestplus scalacheck-1-14 because there is no 1-15 version available for Scalatest 3.1.x but in my experience there are no binary compatibility issues, so it's worth a try. Since this only affects tests the small risk of a crash is acceptable (and problems should be found in CI).

Cheers
~ Felix

@codecov-commenter
Copy link

codecov-commenter commented Jun 17, 2021

Codecov Report

Merging #291 (c18949c) into develop (8e964d9) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head c18949c differs from pull request most recent head 30e13e5. Consider uploading reports for the commit 30e13e5 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #291      +/-   ##
===========================================
+ Coverage    51.78%   51.79%   +0.01%     
===========================================
  Files          311      311              
  Lines        16769    16769              
  Branches       985     1002      +17     
===========================================
+ Hits          8684     8686       +2     
+ Misses        8085     8083       -2     
Impacted Files Coverage Δ
...core/src/main/scala/com/twitter/util/Promise.scala 77.95% <0.00%> (-0.41%) ⬇️
util-core/src/main/scala/com/twitter/io/Buf.scala 93.27% <0.00%> (+0.17%) ⬆️
...in/scala/com/twitter/logging/QueueingHandler.scala 100.00% <0.00%> (+6.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8e964d9...30e13e5. Read the comment docs.

@smarter
Copy link

smarter commented Jun 21, 2021

Upgrading Scalacheck should be fairly simple as I've yet to see any binary compatibility issues between 1.14.x and 1.15.x.

Scalacheck 1.14 and 1.15 are officially binary-compatible yes: https://github.com/typelevel/scalacheck/releases/tag/1.15.0

@tigerlily-he
Copy link
Contributor

Hi @felixbr, Thank you for the PR. I'm currently working on testing the upgrade internally since Twitter works in a monorepo.

finaglehelper pushed a commit to twitter/finagle that referenced this pull request Jun 30, 2021
…4 in sbt

Problem

Scala 3 requires Scalacheck 1.15.x but the sbt builds currently uses 1.14.x

Solution

Upgrade Scalacheck in build.sbt to 1.15.4 to simplify Scala 3 cross-building.

Result

The tests use Scalacheck 1.15.4.  We still use `scalatestplus scalacheck-1-14`
because there is no `1-15` version available for Scalatest 3.1.x.

Closes twitter/util#291
Signed-off-by: Lily He <[email protected]>

JIRA Issues: CSL-11064

Differential Revision: https://phabricator.twitter.biz/D691691
finaglehelper pushed a commit to twitter/scrooge that referenced this pull request Jun 30, 2021
…4 in sbt

Problem

Scala 3 requires Scalacheck 1.15.x but the sbt builds currently uses 1.14.x

Solution

Upgrade Scalacheck in build.sbt to 1.15.4 to simplify Scala 3 cross-building.

Result

The tests use Scalacheck 1.15.4.  We still use `scalatestplus scalacheck-1-14`
because there is no `1-15` version available for Scalatest 3.1.x.

Closes twitter/util#291
Signed-off-by: Lily He <[email protected]>

JIRA Issues: CSL-11064

Differential Revision: https://phabricator.twitter.biz/D691691
finaglehelper pushed a commit to twitter/twitter-server that referenced this pull request Jun 30, 2021
…4 in sbt

Problem

Scala 3 requires Scalacheck 1.15.x but the sbt builds currently uses 1.14.x

Solution

Upgrade Scalacheck in build.sbt to 1.15.4 to simplify Scala 3 cross-building.

Result

The tests use Scalacheck 1.15.4.  We still use `scalatestplus scalacheck-1-14`
because there is no `1-15` version available for Scalatest 3.1.x.

Closes twitter/util#291
Signed-off-by: Lily He <[email protected]>

JIRA Issues: CSL-11064

Differential Revision: https://phabricator.twitter.biz/D691691
finaglehelper pushed a commit to twitter/finatra that referenced this pull request Jun 30, 2021
…4 in sbt

Problem

Scala 3 requires Scalacheck 1.15.x but the sbt builds currently uses 1.14.x

Solution

Upgrade Scalacheck in build.sbt to 1.15.4 to simplify Scala 3 cross-building.

Result

The tests use Scalacheck 1.15.4.  We still use `scalatestplus scalacheck-1-14`
because there is no `1-15` version available for Scalatest 3.1.x.

Closes twitter/util#291
Signed-off-by: Lily He <[email protected]>

JIRA Issues: CSL-11064

Differential Revision: https://phabricator.twitter.biz/D691691
@tigerlily-he
Copy link
Contributor

Thanks again for the PR. This was merged yesterday. I bumped the Scalacheck version for the other libraries as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants