-
Notifications
You must be signed in to change notification settings - Fork 7
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
besom-cfg initial checkin to monorepo #494
Conversation
c4acf15
to
c470137
Compare
985a4d0
to
7ea7860
Compare
@@ -1,6 +1,7 @@ | |||
# Big idea behind using a Justfile is so that we can have modules like in sbt. | |||
|
|||
besom-version := `cat version.txt` | |||
besom-cfg-version := `cat besom-cfg/version.txt` |
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 assume there is a reason behind versioning cfg separately, but wouldn't it make sense to align the versions, just for the sake of simplicity?
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.
no, it wouldn't, this stuff has and has to have a completely separate lifecycle
//> using scala 3.3.3 | ||
|
||
//> using dep com.lihaoyi::os-lib::0.9.3 | ||
//> using dep org.virtuslab::besom-cfg:0.2.0-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.
this will be replaced by current besom version next time you run the version bump script, that's why I was suggesting the version alignment ;)
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.
so there has to be a way to provide exceptions in the version bump script
import scala.quoted.* | ||
import besom.cfg.k8s.syntax.* | ||
|
||
// this is besom-cfg-kubernetes entrypoint |
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.
since this is the entrypoint, I'd strongly advice adding even the most barebones scaladoc :)
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'll strongly consider that once besom-cfg is beyond a prototype
7ea7860
to
ba01946
Compare
89c569f
to
67e46f6
Compare
No description provided.