Skip to content

Conversation

@tanmay-s18
Copy link
Contributor

Migrated build to sbt-typelevel
Updated Scala version to 3.3.0

@samspills samspills self-requested a review March 14, 2025 01:37
build.sbt Outdated
Comment on lines 12 to 13
ThisBuild / crossScalaVersions := Seq("2.12.17", "2.13.11", "3.3.0")
ThisBuild / scalaVersion := "3.3.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's update all of the Scala versions here :)

  • Scala 3 -> 3.3.4
    • This will require bumping Scala Native to the latest version in the 0.4.x line: 0.4.17
  • Scala 2.13 -> 2.13.16
  • Scala 2.12 -> 2.12.20

@@ -1 +1 @@
sbt.version=1.8.2
sbt.version=1.10.7
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh let's also bump our sbt version all the way up to the latest: v1.10.10

Copy link
Contributor

@samspills samspills left a comment

Choose a reason for hiding this comment

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

This is great work, thank you 🎉 🎉

@samspills samspills requested a review from valencik March 17, 2025 17:23
@samspills
Copy link
Contributor

I've also tagged @valencik for review, as he's a co-mentor on the project!

Copy link
Contributor

@valencik valencik left a comment

Choose a reason for hiding this comment

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

Nice work! Looks good to me, just some minor suggestions.

We could maybe add the big reformatting commit to a .git-blame-ignore-revs file so it doesn't show up in git blames. You can see an example of one here:
https://github.com/typelevel/cats/blob/main/.git-blame-ignore-revs

build.sbt Outdated

ThisBuild / crossScalaVersions := Seq("2.12.17", "2.13.8", "3.2.2")
ThisBuild / scalaVersion := "3.2.2"
ThisBuild / crossScalaVersions := Seq("2.12.20", "2.13.16", "3.3.4")
Copy link
Contributor

Choose a reason for hiding this comment

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

Scala 3.3.5 is out now and used elsewhere.

Suggested change
ThisBuild / crossScalaVersions := Seq("2.12.20", "2.13.16", "3.3.4")
ThisBuild / crossScalaVersions := Seq("2.12.20", "2.13.16", "3.3.5")

Copy link
Contributor

@samspills samspills Mar 17, 2025

Choose a reason for hiding this comment

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

We can't update to Scala 3.3.5 because that would force us on to Scala Native 0.5.x and we can't make that bump until cats-effect moves forward there

Copy link
Contributor

Choose a reason for hiding this comment

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

Andrew is right, I think we were just going about the upgrades in an unfortunate order previously. Using Scala Native 0.4.17 and Scala 3.3.5 will work as far as I can figure :)

build.sbt Outdated
ThisBuild / crossScalaVersions := Seq("2.12.17", "2.13.8", "3.2.2")
ThisBuild / scalaVersion := "3.2.2"
ThisBuild / crossScalaVersions := Seq("2.12.20", "2.13.16", "3.3.4")
ThisBuild / scalaVersion := "3.3.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ThisBuild / scalaVersion := "3.3.4"
ThisBuild / scalaVersion := "3.3.5"

@tanmay-s18
Copy link
Contributor Author

added big formatting commit to .git-blame-ignore-revs
Bumped Scala version to 3.3.5
@valencik @samspills

tanmay-s18 and others added 2 commits March 18, 2025 02:41
…ne/CaffeineCache.scala

Co-authored-by: Andrew Valencik <valencik@users.noreply.github.com>
@samspills samspills merged commit d457ee2 into davenverse:main Mar 17, 2025
12 checks passed
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.

3 participants