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

Fixes #1140: Added showVersionless (default true) #1187

Merged
merged 1 commit into from
Nov 24, 2024

Conversation

andrzejj0
Copy link
Contributor

@andrzejj0 andrzejj0 commented Nov 23, 2024

Resolves #1140. New optional parameter, showVersionless default true so that we keep backward compatibility.

Scope of verbose is enhanced: with showVersionless or with processDependencyManagementTransitive, it will also show where the version is managed (groupId:artifactId:version of the POM which manages the version of the dependency).

Please review.

I'm also working on another branch which will use a common service (for dependency updates, dependency updates reports, and possibly for updating those versions as well) to retrieve dependency updates. This will bring cohesion much needed cohesion.

@andrzejj0 andrzejj0 added this to the 2.18.1 milestone Nov 23, 2024
@andrzejj0 andrzejj0 added the bug label Nov 23, 2024
@andrzejj0 andrzejj0 marked this pull request as draft November 24, 2024 06:23
@andrzejj0
Copy link
Contributor Author

Converting to draft: will add this to report as well.

@andrzejj0
Copy link
Contributor Author

I haven't noticed this with reports, so I'll look into them in another PR.

@andrzejj0 andrzejj0 marked this pull request as ready for review November 24, 2024 06:44
pom.xml Outdated Show resolved Hide resolved
showVersionless false filters out dependencies managed outside of the reactor
verbose true adds the source of these dependencies
@andrzejj0 andrzejj0 merged commit 5832dab into mojohaus:master Nov 24, 2024
26 checks passed
@andrzejj0 andrzejj0 deleted the issue-1140 branch November 24, 2024 10:01
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.

Enhancement: limit report to updates actionable in the context of pom.xml
2 participants