-
Notifications
You must be signed in to change notification settings - Fork 1
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
[NU-1790] Add sbt task to publish 1.16 compatibility jars #63
base: staging
Are you sure you want to change the base?
[NU-1790] Add sbt task to publish 1.16 compatibility jars #63
Conversation
6f08b3c
to
34db039
Compare
b499c3e
to
17961ad
Compare
17961ad
to
e480709
Compare
4e6b989
to
a31fe6b
Compare
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 for the review!
a31fe6b
to
ae4b664
Compare
build.sbt
Outdated
val flink116V = "1.16.0" | ||
val currentFlinkV = "1.19.1" | ||
val sttpV = "3.8.11" | ||
val kafkaV = "3.3.1" | ||
val testContainersScalaV = "0.41.0" | ||
|
||
ThisBuild / version := "0.1-SNAPSHOT" | ||
ThisBuild / version := "1.0-nu1.18.0-SNAPSHOT" | ||
|
||
// todo: for now we should regularly bump the version until we start publish single "latest" -SNAPSHOT version | ||
val defaultNussknackerV = "1.18.0-staging-2024-09-24-20698-40fc17dbe-SNAPSHOT" |
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.
why "default"?
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.
Its default because it can be overriden by env variable from CI when testing triggered from main repo. I've moved this into nussknackerV scope to make it more clear.
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.
IMO Nu version should be hardcoded and every time it should be bumped manually (and the change should be reflected with a commit). Like we did in other projects that use Nu as a dependency.
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.
But then triggering tests from core repo would require pushing a commit to nussknacker-flink-compatibility:staging branch with the tag of the released snapshot image instead of triggering an action? Or do you have another solution in mind?
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.
I was thinking about the same solution we have in Quickstart and other Nu-related repos:
- there is an instruction in the Release Management guide to bump the project
- there is a master branch with the latest released Nu version
- there is a staging branch with the snapshot version of the Nu version
- Nu core pipeline triggers this pipeline (for the staging branch) to check if the newest staging version of Nu doesn't cause any problems with this repo
No description provided.