Skip to content

Smoke test mordant against Java LTS releases - #281

Merged
ajalt merged 2 commits into
ajalt:masterfrom
Mahoney-forks:smoke-test-java-lts
Nov 17, 2025
Merged

Smoke test mordant against Java LTS releases#281
ajalt merged 2 commits into
ajalt:masterfrom
Mahoney-forks:smoke-test-java-lts

Conversation

@Mahoney

@Mahoney Mahoney commented Nov 16, 2025

Copy link
Copy Markdown
Contributor

Checks that the basics work against the most recent 3 Java LTS releases, with and
without a TTY (faked with script -q -e -c "bash {0}") and with and
without the --enable-native-access=ALL-UNNAMED flag that Java 24+
requires to avoid warnings.

This currently fails when running on Java 21 with a TTY and
--enable-native-access=ALL-UNNAMED, so I have not made it a
prerequisite for the build passing.

Reference:
#278

Checks that the basics work against all Java LTS releases, with and
without a TTY (faked with `script -q -e -c "bash {0}"`) and with and
without the `--enable-native-access=ALL-UNNAMED` flag that Java 24+
requires.

This currently fails when running on Java 21 with a TTY and
`--enable-native-access=ALL-UNNAMED`, so I have not made it a
prerequisite for the build passing.
- 'docs/**'
- 'samples/**'
- '*.md'
workflow_dispatch:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What does this addition do?

@Mahoney Mahoney Nov 17, 2025

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.

Lets you run the workflow manually via the UI on any branch:
Screenshot 2025-11-17 at 08 43 11

@ajalt
ajalt enabled auto-merge (squash) November 17, 2025 18:12
@ajalt

ajalt commented Nov 17, 2025

Copy link
Copy Markdown
Owner

Thank you!

@ajalt
ajalt merged commit ba8a249 into ajalt:master Nov 17, 2025
6 of 7 checks passed
@Mahoney
Mahoney deleted the smoke-test-java-lts branch November 17, 2025 22:29
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