Skip to content

Fix JavaDoc dist - #51408

Merged
shawkins merged 1 commit into
keycloak:mainfrom
vmuzikar:fix-api-docs-javadocs
Aug 3, 2026
Merged

Fix JavaDoc dist#51408
shawkins merged 1 commit into
keycloak:mainfrom
vmuzikar:fix-api-docs-javadocs

Conversation

@vmuzikar

@vmuzikar vmuzikar commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Closes #51407

I don't have an idea how to test this nicely, we don't currently have similar tests and introducing such a new kind of tests seemed like an overkill to me.

Closes keycloak#51407

Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
@vmuzikar
vmuzikar requested a review from a team as a code owner August 3, 2026 17:43
Copilot AI review requested due to automatic review settings August 3, 2026 17:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes the empty Java API documentation distribution by aligning the assembly with Maven Javadoc’s output path.

Changes:

  • Packages generated Javadocs from target/reports/apidocs.

@shawkins shawkins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@shawkins
shawkins merged commit cb6bd78 into keycloak:main Aug 3, 2026
68 checks passed
@vmuzikar
vmuzikar deleted the fix-api-docs-javadocs branch August 4, 2026 09:13
shawkins pushed a commit that referenced this pull request Aug 6, 2026
Closes #51407


(cherry picked from commit cb6bd78)

Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
alehhu pushed a commit to alehhu/keycloak that referenced this pull request Aug 8, 2026
Closes keycloak#51407

Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
Signed-off-by: alehhu <159355663+alehhu@users.noreply.github.com>
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.

The dist for Java API docs is empty

3 participants