Skip to content

Refactor maven POMs to put all information about the dependencies/plugins in the root POM #940

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

Closed
lburgazzoli opened this issue Oct 19, 2023 · 4 comments · Fixed by #1260
Labels
Milestone

Comments

@lburgazzoli
Copy link

Describe the proposal

As today, the definition of the project dependencies, plugin version and configurations and so on is scattered across the root project pom and its sub-modules which complicates i.e. version management and updates as one has to check if the same version is re-defined in any of the module.

Ideally we can use the root POM file for dependencies and plugins management so each sub-module can have simpler references to the artifacts and plugins.

lburgazzoli added a commit to lburgazzoli/dapr-java-sdk that referenced this issue Oct 19, 2023
@lburgazzoli
Copy link
Author

I've done a first POC here: https://github.com/lburgazzoli/dapr-java-sdk/tree/gh-940 in case this make sense

@artursouza
Copy link
Contributor

LGTM. I can accept a PR on this.

@artursouza artursouza added this to the v1.11 milestone Nov 13, 2023
@mukundansundar mukundansundar modified the milestones: v1.11, v1.12 Feb 8, 2024
@artursouza artursouza modified the milestones: v1.12, v1.13 Jul 23, 2024
@artursouza artursouza modified the milestones: v1.13, v1.14 Oct 22, 2024
@cicoyle
Copy link
Contributor

cicoyle commented Jan 31, 2025

Hey @lburgazzoli - were you able and willing to make a PR for the changes from your branch linked above?

@cicoyle cicoyle modified the milestones: v1.14, v1.15 Jan 31, 2025
@lburgazzoli
Copy link
Author

@cicoyle unfortunately not, I've been sidetracked to other activities. not sure if I'll ever have time to resume that work :(

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

Successfully merging a pull request may close this issue.

4 participants