Skip to content

Update CI actions to latest major versions, upload .jar - #106

Merged
lognaturel merged 1 commit into
getodk:masterfrom
lindsay-stevens:update-ci
Oct 30, 2026
Merged

lognaturel merged 1 commit into
getodk:masterfrom
lindsay-stevens:update-ci

Conversation

@lindsay-stevens

Copy link
Copy Markdown
Contributor
  • older actions versions now showing warnings on Actions run history that they are running with a newer version of Node (v24) than they are written for (v20). (setup-java still supports java v8)
  • the "Upload libs artifact" uploads the "build/libs" as a zip but it (should) only contain the release jar file, so upload only the jar file to simplify release prep. (the pyxform Validate updater gets release assets from the release but not from actions)

What has been done to verify that this works as intended?

Checked actions run on my fork (run here) and it looks like the below screenshot - here the jar uses the last commit hash but for releases it uses the last tag:

image

Why is this the best possible solution? Were any other approaches considered?

Considered adding a release workflow, to automatically attach the jar to a tagged release. But it seems like at the moment, that requires creating a GitHub personal access token or changing permissions on the org token (or creating a GitHub app) etc - in any case it's an extra thing to manage vs. the manual method done about once a year (download built artifact, upload to release page). So, it's possible but I wasn't sure enough that it was worth it.

Are there any risks to merging this code? If so, what are they?

Unlikely to ever be a problem but when archive: false only one file is allowed so if the build emits more than one jar it wouldn't work.

- older actions versions now showing warnings on Actions
  run history that they are running with a newer version
  of Node (v24) than they are written for (v20).
- the "Upload libs artifact" uploads the "build/libs" as
  a zip but it (should) only contain the release jar file,
  so upload only the jar file to simplify release prep.

@lognaturel lognaturel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

Agreed uploading the artifact manually is ok.

@lognaturel
lognaturel merged commit af77c80 into getodk:master Oct 30, 2026
1 check passed
@lindsay-stevens
lindsay-stevens deleted the update-ci branch November 2, 2026 07:24
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