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

Rename bom -> platform #29

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Rename bom -> platform #29

merged 1 commit into from
Nov 24, 2022

Conversation

melix
Copy link
Collaborator

@melix melix commented Nov 23, 2022

The name "bom" was confusing with regards to the old BOM and the new "core" BOM. We will now refer to the BOM which aggregates several Micronaut modules (including core), driving releases, as the "platform BOM".

Each module should have its own BOM, included in this project, and this is where the "core BOM" is included.

This will require both updates to the Micronaut Build plugins and Micronaut Gradle plugin.

The repository should also be renamed for clarity.

The name "bom" was confusing with regards to the old BOM and the
new "core" BOM. We will now refer to the BOM which aggregates
several Micronaut modules (including core), driving releases, as
the "platform BOM".

Each module should have its own BOM, included in this project, and
this is where the "core BOM" is included.
@melix melix added this to the 4.0.0 milestone Nov 23, 2022
@melix melix self-assigned this Nov 23, 2022
@melix
Copy link
Collaborator Author

melix commented Nov 23, 2022

I think the static analysis error comes from the fact the project isn't found in Sonar anymore because of the rename /cc @alvarosanchez

@alvarosanchez
Copy link
Member

The Sonar project is linked to a repo, not to a Gradle module. I still see the project at https://sonarcloud.io/project/overview?id=micronaut-projects_micronaut-bom

If it keeps failing after merging this, I can delete and re-import it.

@melix melix merged commit 9ed75ad into master Nov 24, 2022
@melix melix deleted the cc/rename-to-platform branch November 24, 2022 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants