Skip to content

fake#1

Open
jonahgraham wants to merge 8 commits intomainfrom
fake
Open

fake#1
jonahgraham wants to merge 8 commits intomainfrom
fake

Conversation

@jonahgraham
Copy link
Copy Markdown
Owner

No description provided.

sebthom and others added 8 commits November 14, 2025 15:47
- update URLs to reference documentation
- test on MacOS ARM and Intel
- update versions of github actions (checkout, setup-java)
- parameterize validate job so that in case of errors an SSH debug
session can be opened

Co-authored-by: Jonah Graham <[email protected]>
…and 21

- on CI build with Java 21, test with 11, 17 and 21
- on Dev machine, build can also be 17
- Decouple JDK to run Gradle from the JDK used to compile/test.
- Use Java toolchains to select JDK 11 for compilation and test
- Set javac --release 11 for consistent API/bytecode.
- Launch Test tasks with the JDK 11 toolchain for non-xtend cases
- Launch test tasks with JDK 17, 21 for all tests
- Update Tycho to newer version that works with Java 21

Co-authored-by: Jonah Graham <[email protected]>
The results are published, but can be seen in build summary.
For some of the trivial use cases of xtend change them to pure Java.
This allows more of the tests to build and run with Java 11 as
xtend now requires Java 17.
xtend converts `.last` to `List.getLast()` if the JVM it is running
under has it, so the generated java files will use `List.getLast()`
even though we want to target older JVM. `List.getLast()` was introduced
in Java 21, which is newer than our supported version of Java11.

This is the workaround recommended in eclipse-xtext/xtext#3535
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 14, 2025

Test Results

  512 files  ±0    512 suites  ±0   46s ⏱️ +5s
  363 tests ±0    363 ✅ ±0  0 💤 ±0  0 ❌ ±0 
4 100 runs  ±0  4 100 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit cf051ce. ± Comparison against base commit 456ac0f.

♻️ This comment has been updated with latest results.

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