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

Upgrade Maven Javadoc Plugin from 3.0.1 to 3.1.0 #3478

Conversation

WilliamCollishaw
Copy link
Contributor

@WilliamCollishaw WilliamCollishaw commented May 21, 2019

Recreated PR from #3476
Upgrade Maven Javadoc Plugin from 3.0.1 to 3.1.0
Additionally added wildcard syntax to the list of excluded package names due to failing without it after the upgrade

Comment from @njhill in PR #3370 notes a javadoc plugin issue with JDK 11 that should be resolved in this most recent version.

The functionality as described in the excludePackageNames documentation does not work as explained which did previously work in 3.0.1. I raised MJAVADOC-605, which duplicated MJAVADOC-584 explaining the issue and have included a work around for the issue which significantly increased the size of the excludePackageNames list.

Additionally added wildcard syntax to the list of excluded package names due to failing without it after the upgrade
@netdpb
Copy link
Member

netdpb commented May 23, 2019

Cool. Merging internally.

@cpovirk
Copy link
Member

cpovirk commented May 23, 2019

Yes, thank you!

@qingliaowu
Copy link

Cool

@ronshapiro ronshapiro mentioned this pull request May 28, 2019
ronshapiro pushed a commit that referenced this pull request May 28, 2019
Fixes #3478.

RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=249642506
ronshapiro pushed a commit that referenced this pull request May 28, 2019
Fixes #3478.

RELNOTES=n/a

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=249642506
@WilliamCollishaw WilliamCollishaw deleted the Upgrade-maven-javadoc-plugin-v2 branch May 28, 2019 15:02
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.

5 participants