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 for reports #1188

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

andrzejj0
Copy link
Contributor

I managed to reproduce #1140 for reports, so here is a fix.

Added an integration test + showVersionless for dependency-updates-report.

Added some small refactoring.

@andrzejj0 andrzejj0 added the bug label Nov 24, 2024
@andrzejj0 andrzejj0 added this to the 2.18.1 milestone Nov 24, 2024
@andrzejj0 andrzejj0 self-assigned this Nov 24, 2024
@andrzejj0 andrzejj0 force-pushed the dependency-report-issue-1140 branch from 79e312c to 2f41376 Compare November 26, 2024 12:18
@andrzejj0 andrzejj0 force-pushed the dependency-report-issue-1140 branch from 2f41376 to e0ba00f Compare November 27, 2024 06:47
* <p>This is similar to {@code processDependencyManagementTransitive}, but will
* report updates on dependencies.</p>
*
* @since 2.18.1
Copy link
Member

Choose a reason for hiding this comment

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

Maybe should be a 2.19.0 as we add new feature ... what do you think?

Copy link
Member

Choose a reason for hiding this comment

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

or it is for fixing bug only ...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. Changed that in DisplayDependencyUpdatesMojo as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well, a bug fix and a feature toggle. I don't know :-)

Added an integration test + showVersionless for dependency-updates-report
@andrzejj0 andrzejj0 force-pushed the dependency-report-issue-1140 branch from e0ba00f to 72a4819 Compare November 27, 2024 06:56
@andrzejj0 andrzejj0 merged commit e454b4f into mojohaus:master Nov 28, 2024
26 checks passed
@andrzejj0 andrzejj0 deleted the dependency-report-issue-1140 branch November 28, 2024 12:15
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 this pull request may close these issues.

2 participants