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
#24752 Fix timestamped versions #24845
Conversation
project/AkkaBuild.scala
Outdated
@@ -20,13 +20,15 @@ object AkkaBuild { | |||
|
|||
lazy val buildSettings = Dependencies.Versions ++ Seq( | |||
organization := "com.typesafe.akka", | |||
version := "2.5-SNAPSHOT") | |||
version := (version in ThisBuild).value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you document this a bit more? It seems rather magical where the actual value comes from now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @2m comment and merge? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. But with sbt inspect
is the king. It shows where values for settings come from more robustly than comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
document intent why we do this so weirdly, and hint at inspect?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment
Test PASSed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -20,13 +20,16 @@ object AkkaBuild { | |||
|
|||
lazy val buildSettings = Dependencies.Versions ++ Seq( | |||
organization := "com.typesafe.akka", | |||
version := "2.5-SNAPSHOT") | |||
// use the same value as in the build scope, so it can be overriden by stampVersion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this helps, thanks! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, LGTM
Test PASSed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
Ideally we should use sbt-dynver, but currently version tagged commits are not part of the direct history of master. Upon release we fork master and add a commit which only has version change. Then we tag that commit with a release tag. Therefore sbt-dynver can not currently derive version from history.
Fixes #24752