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

Build and publish only LTS scala 3 #452

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Build and publish only LTS scala 3 #452

merged 2 commits into from
Feb 12, 2025

Conversation

rbscgh
Copy link
Contributor

@rbscgh rbscgh commented Feb 7, 2025

declaring 3 scala minor versions always takes the latest when projects pull in the dependency. this causes issue as the TASTy reader is always looking at the latest binary (in this 3.6.3) so runtimes cause incompatibility errors. if developers wish to use 3.6.3, twinagle will still compile for them when its built with 3.3.x.

This also reduces some complexity in the tests configuration as we dont need to manage 2 versions of scalamock

also handles #451

@rbscgh rbscgh requested review from ccmtaylor and a team February 7, 2025 11:16
@@ -2,8 +2,7 @@ import sbt.CrossVersion

lazy val scala212 = "2.12.18"
lazy val scala213 = "2.13.14"
lazy val scala3LTS = "3.3.4"
lazy val scala3 = "3.6.3"
lazy val scala3LTS = "3.3.5"
Copy link
Member

Choose a reason for hiding this comment

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

I'm not too familiar with scala3 release process / backwards compat. When will we know. to update this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so they backported random things into 3.3.4 which i need to be able to ignore specific warnings from src dirs.. its hard to say basically.. 3.3.5 only came out in late jan

)
case _ => Seq.empty
}
},
// compile protobuf messages for unit tests
Project.inConfig(Test)(sbtprotoc.ProtocPlugin.protobufConfigSettings),
Test / scalacOptions += {
CrossVersion.partialVersion(scalaVersion.value) match {
case Some((3, minor)) if minor > 3 => "-experimental"
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if we'll run into this when @scala-steward bumps the version of scalamock / scala 3 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good q 🤷🏾‍♂️

@rbscgh rbscgh merged commit 99cbc18 into master Feb 12, 2025
1 check passed
@rbscgh rbscgh deleted the LTS-only branch February 12, 2025 13:19
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.

2 participants