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

[AMLII-1828] Cleaning up pom.xml #529

Merged
merged 8 commits into from
Jul 12, 2024
Merged

Conversation

carlosroman
Copy link
Contributor

@carlosroman carlosroman commented Jun 10, 2024

Cleaned up the pom.xml and bumped some of the dependencies. Deployments still work after all the changes (see SNAPSHOT here).

@carlosroman carlosroman force-pushed the carlosroman/AMLII-1828-pom-tidy branch from 5fa6ad9 to ba7b648 Compare June 10, 2024 11:28
@carlosroman carlosroman force-pushed the carlosroman/AMLII-1828-pom-tidy branch from 3c45d29 to 171f467 Compare June 19, 2024 11:03
@carlosroman carlosroman force-pushed the carlosroman/AMLII-1828-pom-tidy branch from 7d1968c to 311e4a6 Compare July 10, 2024 13:33
@carlosroman carlosroman force-pushed the carlosroman/AMLII-1828-pom-tidy branch from 750e9c1 to 83b5134 Compare July 10, 2024 15:37
@carlosroman carlosroman marked this pull request as ready for review July 12, 2024 08:59
@carlosroman carlosroman requested a review from a team as a code owner July 12, 2024 08:59
@@ -95,7 +100,7 @@
<plugin>
<groupId>org.sonatype.plugins</groupId>
<artifactId>nexus-staging-maven-plugin</artifactId>
<version>1.6.13</version>
<version>${nexus-staging-maven-plugin.version}</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, what is the advantage of specifying the version via a property?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went back and forth between specifying them in property or just moving them out of property completely or moving it all to dependency-management section. I was aiming for some consistency while I went through and upgraded some of the plugins and dependencies as we had mixture.

@carlosroman carlosroman merged commit 068d0e7 into master Jul 12, 2024
9 checks passed
@carlosroman carlosroman deleted the carlosroman/AMLII-1828-pom-tidy branch July 12, 2024 15:36
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.

2 participants