-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
The client bom with dependencies should include kubernetes-model #924
Comments
Hey @iocanel , help me out here please! ( New to Maven ecosystem. ) However, in the repo's case, the BOM specification in the parent POM doesn't mention a version and just sets a * regex ( https://github.com/fabric8io/kubernetes-client/blob/master/pom.xml#L239 ) and then, modules kubernetes-client and openshift-client pom have to specify the direct dependency versions in their pom ( https://github.com/fabric8io/kubernetes-client/blob/master/kubernetes-client/pom.xml#L165 ). Doesn't this defeats the purpose of using a BOM? I'm confused here. Please correct me if I'm wrong. |
@dev-gaur : I think this bom is not something used by our project. We're providing this bom to be used by projects who have our project as a dependency in their pom, so that they don't have to worry about dependency version management. |
@rohanKanojia You are right, the idea of bom is that you define deps of your own project to be consumed by other projects (not yourself although it is possible as well). The idea of bom is to simplify how others are going to use your library. Let's take a simple example with Spring Boot. Spring Boot has several modules and not all of them are required at once, for example you could have What you know is that So to have some way to lightly force this, Spring-Boot provides a bom. So you only need to register the bom in your project, and then in any dependency that you added that is specified in that bom, you don't need to set any version since it is taken from bom. |
No description provided.
The text was updated successfully, but these errors were encountered: