-
Notifications
You must be signed in to change notification settings - Fork 16
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
[WIP] Initial proposal for a Spring Boot parent #66
Conversation
Yeah, we would probably need to update Atomist but it shouldn't be much of an effort |
Nice work @gytis . LGTM at the moment, the only remark I would do and it's very minor is I don't like the name of the module |
@aureamunoz in principal I agree. But the idea here was that this |
Alternatively, Erin suggests to have the same |
Ok, thanks @gytis for the explanation, it's what I was thinking right now that from the point of view of users that will be coherent :-) |
I know I would be confused to have the BOM as a parent :) |
<dependency> | ||
<groupId>me.snowdrop</groupId> | ||
<artifactId>spring-boot-bom</artifactId> | ||
<version>2.1.6-SNAPSHOT</version> |
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.
Can't we use ${project.version} here?
<packaging>pom</packaging> | ||
|
||
<properties> | ||
<fabric8-maven-plugin.version>4.1.0</fabric8-maven-plugin.version> |
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.
Tangent to the core issue but is this version tested? See discussion in snowdrop/rest-http-example#111
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.
Though, I guess, this change would address the need to update all examples so that they are all consistent.
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.
For now I wanted to get the opinion on the general structure. All the values are taken from original spring-boot-bom or latest http booster.
I'd prefer to use parent in the name if the purpose is for that pom to be used as a parent to applications. BOMs are conventionally used in dependency management sections so I think it'd be confusing to use bom in the name for a parent POM… 😅 |
So we could go two ways. Multiple modules as in this PR or renaming |
Also while we're on a subject of reorganising. What's the situation with #37? |
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.
Shouldn't we import the dekorate bom too?
I've put together an initial proposal for a possible Spring Boot parent addition. I've restructured the project in a following way:
spring-boot-build
- project base containing main repositories and release profile needed for all modulesspring-boot-bom
- the originalspring-boot-bom
but now only containing dependencies and inheriting fromspring-boot-build
spring-boot-parent
- inherits fromspring-boot-build
and includesspring-boot-bom
in itsdependencyManagement
section. For the user convenience it also includesspring-boot-maven-plugin
andfabric8-maven-plugin
as well as resource filtering. What plugins and configurations should be included here is up for discussion.This should be backwards compatible with the existing users as
spring-boot-bom
coordinates did not change and only parts of it moved to its parent (spring-boot-build
). This might, however, impede our release process in some unexpected way in case there are some hardcoded values in the scripts. Thus, I don't recommend moving ahead with this for the 2.1.10 release but start reorganisation immediately after it.