Skip to content

Github Action to Build and Publish Releases - #435

Merged
Felix-Griesau-SAP merged 3 commits into
mainfrom
add-central-release-workflow
Sep 15, 2026
Merged

Felix-Griesau-SAP merged 3 commits into
mainfrom
add-central-release-workflow

Conversation

@Felix-Griesau-SAP

@Felix-Griesau-SAP Felix-Griesau-SAP commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

This PR adds a github workflow to build and publish releases.

It will be triggered automatically upon creating a github release, or independently on demand. I added a little md file explaining some details, that should also help reproducing the setup if it ever breaks.

As of now, I haven't created the proper credentials yet. As already discussed, my plan is to first test-run this (after merge) without valid credentials to see if everything is behaving as expected, and then testing with valid credentials.

@Felix-Griesau-SAP
Felix-Griesau-SAP marked this pull request as ready for review September 14, 2026 15:40

@KarstenSchnitter KarstenSchnitter 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.

Thank you for providing this PR. I have a suggestion on the private key passphrase and some minor formulation question.

Comment thread .github/workflows/RELEASE.md Outdated
Comment thread .github/workflows/RELEASE.md Outdated
- Doc wording
- GPG passphrase support

@KarstenSchnitter KarstenSchnitter 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.

Thanks for the effort. Looks good to me. Happy testing.

@Felix-Griesau-SAP
Felix-Griesau-SAP merged commit c0c9467 into main Sep 15, 2026
3 checks passed
@Felix-Griesau-SAP
Felix-Griesau-SAP deleted the add-central-release-workflow branch September 15, 2026 13:19
@KarstenSchnitter

Copy link
Copy Markdown
Contributor

@Felix-Griesau-SAP congratulations to your first merge.

- name: Set up JDK, Maven settings, GPG
uses: actions/setup-java@cf277c60eb25467037889841efdb72551f06f6c3 # v4.9.1
with:
distribution: temurin

@aamotharald aamotharald Sep 18, 2026

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.

@KarstenSchnitter technically it makes no difference, but I would favor a sapmachine here . FYI @Felix-Griesau-SAP

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.

So far I have built the releases using a SAPMachine. I support the recommendation.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll open another PR for that.
I based the choice on the existing configuration in the other workflow though. I'll change that as well then:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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.

3 participants