Skip to content

Fix issue 367#372

Merged
cstamas merged 2 commits into
masterfrom
issue-367
Jun 30, 2026
Merged

Fix issue 367#372
cstamas merged 2 commits into
masterfrom
issue-367

Conversation

@cstamas

@cstamas cstamas commented Jun 9, 2026

Copy link
Copy Markdown
Member

Sadly, I did not see #369

These two should be merged/melded into one PR, as the other PR adds nice UT.

Fixes apache/maven#12181

@cstamas cstamas requested a review from slawekjaranowski June 9, 2026 21:14

// used deprecated methods - FIXME in DescribeMojo
if (mavenVersion.startsWith('4.') || mavenVersion.startsWith('3.10.')) {
if (mavenVersion.startsWith('4.')) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

interesting how without it #369 will pass ...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see no 3.10 on GH 😄

@slawekjaranowski

Copy link
Copy Markdown
Member

so maybe merge this first, and next rebase and merg unit test from #369

Comment thread src/main/java/org/apache/maven/plugins/help/DescribeMojo.java
@cstamas cstamas self-assigned this Jun 25, 2026
@cstamas cstamas added the bug Something isn't working label Jun 25, 2026
@cstamas cstamas merged commit 2976b29 into master Jun 30, 2026
15 checks passed
@cstamas cstamas deleted the issue-367 branch June 30, 2026 10:13
@github-actions github-actions Bot added this to the 3.5.2 milestone Jun 30, 2026
@pzygielo

pzygielo commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

So... is

fixed or not?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Describe deprecated methods in org.apache.maven.lifecycle

3 participants