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

Use maven javadoc for pde #1501

Merged

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Nov 2, 2023

Currently we use a hand crafted options file to build the javadoc files
what is cumbersome and error prone.

This aims to replace the ant calls by the maven-javadoc-plugin so that
dependencies declared in the pom are automatically passed to javadoc
build. Beside that this makes the doc used for referenced links is added
as a dependency and then unpacked to reduce the need to specify absolute
path to other projects in the reactor.

@laeubi laeubi force-pushed the use_maven_javadoc_for_pde branch 2 times, most recently from 878b42a to 031d869 Compare November 3, 2023 05:55
Currently we use a hand crafted options file to build the javadoc files
what is cumbersome and error prone.

This aims to replace the ant calls by the maven-javadoc-plugin so that
dependencies declared in the pom are automatically passed to javadoc
build. Beside that this makes the doc used for referenced links is added
as a dependency and then unpacked to reduce the need to specify absolute
path to other projects in the reactor.
@laeubi laeubi force-pushed the use_maven_javadoc_for_pde branch from 031d869 to 3529a9e Compare November 3, 2023 07:29
@laeubi laeubi marked this pull request as ready for review November 3, 2023 07:29
@laeubi laeubi requested a review from akurtakov November 3, 2023 07:29
@akurtakov
Copy link
Member

I'm merging this one and starting I-build now so we can verify the docs are in good shape.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants