Skip to content

Conversation

@ipa-rwu
Copy link
Member

@ipa-rwu ipa-rwu commented Oct 2, 2025

@ipa-nhg
This pull request upgrades the Java version used across the project from Java 19 (and in some places Java 11/1.8) to Java 21. It also updates related build tools and dependencies to ensure compatibility with Java 21. The changes affect build configuration files, Maven POMs, Eclipse project settings, and GitHub Actions workflows.

Key changes include:

Java Version Upgrade:

  • Updated all Eclipse .classpath files and .settings/org.eclipse.jdt.core.prefs to use JavaSE-21 instead of JavaSE-19 or earlier. [1] [2] [3] [4] [5]
  • Changed Bundle-RequiredExecutionEnvironment in MANIFEST.MF files for plugins to JavaSE-21. [1] [2]

Build and Dependency Updates:

  • Updated Maven compiler source/target versions to 21 in pom.xml files, and upgraded related plugin and dependency versions (xtextVersion, tycho-version, mwe2Version). [1] [2]
  • Set the Tycho build execution environment to JavaSE-21 and updated the Eclipse release repository URL to a newer version. [1] [2]
  • Updated the GitHub Actions workflow to use Java 21 with the Temurin distribution, and enabled Maven caching.

Other minor changes:

  • Minor formatting and comment improvements in configuration files.
  • Removed trailing whitespace in documentation and license files. [1] [2]

@ipa-rwu ipa-rwu force-pushed the rwu/fix/ci branch 3 times, most recently from 62add0f to 1d926ce Compare October 6, 2025 07:48
- Updated the generated files to reflect the new Xtext version (2.39.0).
- Changed Java SE version from 19 to 21 in .classpath and .settings/org.eclipse.jdt.core.prefs.
- Updated MANIFEST.MF to require Java SE 21.
- Adjusted the expected size of the injectors map in XtextActivator.
- Modified pom.xml to use the variable for MWE2 version.
- Cleaned up whitespace and formatting in various generator files for consistency.
- Added a new utility method to remove leading slashes in GeneratorHelpers.
@ipa-rwu ipa-rwu changed the title Fix CI: upgrade to JDK21 Upgrade to JDK21 Oct 6, 2025
@ipa-nhg ipa-nhg merged commit 4edb86b into ipa320:main Oct 22, 2025
1 check passed
@ipa-nhg
Copy link
Member

ipa-nhg commented Oct 22, 2025

@ipa-rwu Wow, many thanks for fixing this CI system 👏🏽 👏🏽
I usually do not clean the spaces (formatting issue) because it breaks again with every new Xtext automatic generation of the sources. I will merge now, but I don't know if this is maintainable.

@ipa-rwu
Copy link
Member Author

ipa-rwu commented Oct 22, 2025

@ipa-rwu Wow, many thanks for fixing this CI system 👏🏽 👏🏽 I usually do not clean the spaces (formatting issue) because it breaks again with every new Xtext automatic generation of the sources. I will merge now, but I don't know if this is maintainable.

I usually run pre-commit to format before pushing. https://github.com/ipa320/RosTooling/blob/main/.pre-commit-config.yaml

@ipa-rwu ipa-rwu deleted the rwu/fix/ci branch October 22, 2025 12:06
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