Skip to content
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

Optimizations in Spring Modulith dependency setup causes BOM not being added #1490

Closed
wants to merge 1 commit into from

Conversation

odrotbohm
Copy link
Contributor

The removal of the core dependency causes the BOM not being added as that's contained in the dependency declaration. Manually adding the BOM (build.boms().add(…)) adds the BOM segment but does not declare the version property that's needed to resolve it properly.

The removal of the core dependency causes the BOM not being added as that's contained in the dependency declaration. Manually adding the BOM (build.boms().add(…)) adds the BOM segment but does not declare the version property that's needed to resolve it properly.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 28, 2024
@mhalbritter mhalbritter added type: bug and removed status: waiting-for-triage An issue we've not yet triaged labels May 28, 2024
@mhalbritter mhalbritter self-assigned this May 28, 2024
@mhalbritter mhalbritter changed the title Revert optimizations in Spring Modulith dependency setup. Revert optimizations in Spring Modulith dependency setup May 28, 2024
@mhalbritter mhalbritter changed the title Revert optimizations in Spring Modulith dependency setup Optimizations in Spring Modulith dependency setup causes BOM not being added May 28, 2024
mhalbritter pushed a commit that referenced this pull request May 28, 2024
The removal of the core dependency causes the BOM not being added as
that's contained in the dependency declaration. Manually adding the BOM
(build.boms().add(…)) adds the BOM segment but does not declare the
version property that's needed to resolve it properly.

See gh-1490
@mhalbritter
Copy link
Contributor

Thanks Ollie!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants