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

allow Scala nightlies via system properties #1865

Merged
merged 1 commit into from
Sep 10, 2023

Conversation

bjaglin
Copy link
Collaborator

@bjaglin bjaglin commented Sep 10, 2023

@bjaglin
Copy link
Collaborator Author

bjaglin commented Sep 10, 2023

cc @SethTisue

@bjaglin bjaglin force-pushed the upstream-communitybuild-sysprod branch from 081cb67 to bbc480c Compare September 10, 2023 11:08
@@ -8,7 +8,7 @@ import scala.util.Try

object Dependencies {
val scala212 = "2.12.18"
val scala213 = "2.13.12"
val scala213 = sys.props.getOrElse("scala.nightly", "2.13.12")
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

not sure how scala-steward will handle that, but since the bump is manual during the embargo, it does not matter much

@bjaglin bjaglin force-pushed the upstream-communitybuild-sysprod branch from bbc480c to acbee63 Compare September 10, 2023 11:27
@bjaglin bjaglin changed the title allow Scala 2.13 nightlies via system property allow Scala nightlies via system properties Sep 10, 2023
val scala213 = "2.13.12"
val scala3 = "3.3.1"
val scala212 = sys.props.getOrElse("scala212.nightly", "2.12.18")
val scala213 = sys.props.getOrElse("scala213.nightly", "2.13.12")
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@SethTisue I have suffixed the system property with the scala version to make it possible to tweak any scala version, I assume this is not a problem as it's explicitly forwarded there so it can easily be changed?

Copy link
Contributor

Choose a reason for hiding this comment

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

yup that's fine

@bjaglin bjaglin marked this pull request as ready for review September 10, 2023 11:29
@bjaglin bjaglin merged commit 3da14e0 into scalacenter:main Sep 10, 2023
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