Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[MJAVADOC-723] Upgrade Maven Reporting API to 3.1.1/Complete with Maven Reporting Impl 3.2.0 #158

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented Aug 7, 2022

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@pzygielo pzygielo marked this pull request as ready for review August 7, 2022 11:12
…en Reporting Impl 3.2.0

This not only upgrades to Maven Reporting API 3.1.1, but also logically
completes this plugin with bits from Maven Reporting Impl 3.2.0 because
we cannot inherit from AbstractMavenReport since Java does not support
multiple inheritance. It should now behave as if using m-r-impl.

Co-authored-by: Piotrek Żygieło <pzygielo@users.noreply.github.com>

This closes apache#158
@michael-o
Copy link
Member

Folks, eyes please. This should now do it. Everything passes and it should now be like using m-r-impl. Maven JXR passes with this.

@pzygielo
Copy link
Contributor Author

pzygielo commented Aug 8, 2022

I confirm that on my side - plugin with such change (3.4.1-SNAPSHOT a10f0b1) works fine when forked by m-site-plugin:3.12.1

Thanks!

@michael-o michael-o changed the title [MJAVADOC-723] Upgrade Maven Reporting API to 3.1.1 [MJAVADOC-723] Upgrade Maven Reporting API to 3.1.1/Complete with Maven Reporting Impl 3.2.0 Aug 9, 2022
@michael-o
Copy link
Member

Guys, please check. This is a blocker.

@asfgit asfgit closed this in a10f0b1 Aug 10, 2022
@asfgit asfgit merged commit a10f0b1 into apache:master Aug 10, 2022
@pzygielo pzygielo deleted the MJAVADOC-723 branch August 10, 2022 20:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants