Skip to content

Db contrib/gha build javadoc restapi#66

Open
gaborherr-db wants to merge 2 commits intofinos:mainfrom
deutschebank:db-contrib/gha-build-javadoc-restapi
Open

Db contrib/gha build javadoc restapi#66
gaborherr-db wants to merge 2 commits intofinos:mainfrom
deutschebank:db-contrib/gha-build-javadoc-restapi

Conversation

@gaborherr-db
Copy link
Contributor

This adds building of javadocs and openapi generation with GHA. I am not sure if it works, but it is hard to test from here. Any fixes will be appreciated.

…eration steps.

Merge in FLUXNOVA/fluxnova-bpm-platform from feature/gha-build-javadoc-restapi to db-feature/gha-build-javadoc-restapi

* commit '7d8003b819d8068fa9040a7aa07127a2c80dab3b':
  Extended workflow with Javadocs and openAPI generation steps.
pieter-schutte
pieter-schutte previously approved these changes Jan 22, 2026
@gliptak
Copy link
Contributor

gliptak commented Jan 23, 2026

https://github.com/finos/fluxnova-bpm-platform/actions/runs/21280546449/job/61249254191?pr=66

Error:  Failed to execute goal org.sonatype.central:central-publishing-maven-plugin:0.7.0:publish (injected-central-publishing) on project fluxnova-wildfly26-modules: org.apache.maven.artifact.deployer.ArtifactDeploymentException: Failed to retrieve remote metadata org.finos.fluxnova.bpm:fluxnova-engine-cdi:1.1.0-SNAPSHOT/maven-metadata.xml: Could not transfer metadata org.finos.fluxnova.bpm:fluxnova-engine-cdi:1.1.0-SNAPSHOT/maven-metadata.xml from/to fluxnova-bpm-platform (https://maven.pkg.github.com/finos/fluxnova-bpm-platform): authentication failed for https://maven.pkg.github.com/finos/fluxnova-bpm-platform/org/finos/fluxnova/bpm/fluxnova-engine-cdi/1.1.0-SNAPSHOT/maven-metadata.xml, status: 401 Unauthorized -> [Help 1]

@deutschebank-sync deutschebank-sync force-pushed the db-contrib/gha-build-javadoc-restapi branch from 6eb8814 to 96607ff Compare January 29, 2026 13:01

- name: Generate Javadoc
run: |
mvn -B -e package javadoc:javadoc javadoc:aggregate \
Copy link
Contributor

Choose a reason for hiding this comment

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

@gaborherr-db Thanks for adding this java doc generate action in release workflow. This is much needed change that we have been talking about. Were you guys able to do end to end testing?

with:
github_token: ${{ secrets.GITHUB_TOKEN }}
publish_dir: ./engine-rest/engine-rest-openapi/target/generated-source/openapi-json
destination_dir: openapi/${{ needs.publish-central.outputs.minor_version }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we know where we’ll host it after the pages are generated and moved to openapi/* folder??

Copy link
Contributor

@prajwolbhandari1 prajwolbhandari1 left a comment

Choose a reason for hiding this comment

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

Looks good to me other than few questions

@pieter-schutte
Copy link
Contributor

Approved on the assumption that your upstream PRs will resolve the build issues

@RachanaRamshetty88 RachanaRamshetty88 moved this from Backlog to In review in Fluxnova Project Board Feb 19, 2026
@gliptak
Copy link
Contributor

gliptak commented Feb 26, 2026

@gaborherr-db please resolve conflicts (and it will also trigger a build)

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

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

4 participants