Skip to content

Toolchain support for test plugins in devmode#1957

Merged
venmanyarun merged 15 commits into
OpenLiberty:mainfrom
sajeerzeji:toolchain_test_plugings
Jan 28, 2026
Merged

Toolchain support for test plugins in devmode#1957
venmanyarun merged 15 commits into
OpenLiberty:mainfrom
sajeerzeji:toolchain_test_plugings

Conversation

@sajeerzeji

@sajeerzeji sajeerzeji commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Added toolchain support for test plugins(maven-surefire-plugin and maven-failsafe-plugin) in devmode.

Test plugins(maven-surefire-plugin and maven-failsafe-plugin) will now use toolchain level jdk version. If the user have a different jdk toolchain configuration for the test plugin, we will show the warning as well (Priority goes to our plugins toolchain).

Bumped liberty maven plugin version to 3.12.0

@sajeerzeji sajeerzeji requested a review from venmanyarun January 7, 2026 09:18
@sajeerzeji sajeerzeji requested a review from cherylking January 7, 2026 10:58
Comment thread liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/server/DevMojo.java Outdated
@venmanyarun

venmanyarun commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

@sajeerzeji
Please fix the copyright years

@sajeerzeji

Copy link
Copy Markdown
Contributor Author

@sajeerzeji Please fix the copyright years

Updated the copyright years

Comment thread liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/server/DevMojo.java Outdated
Comment thread README.md Outdated
@sajeerzeji sajeerzeji requested a review from cherylking January 20, 2026 12:25
Comment thread docs/toolchain.md Outdated
Sajeer and others added 4 commits January 21, 2026 16:35
…r understanding and updated with a reference document
…iguration mismatches of Test Plugins/Compiler Plugin vs liberty
…ation section as per the lack of documentations available
@venmanyarun venmanyarun merged commit a2584ed into OpenLiberty:main Jan 28, 2026
23 of 24 checks passed
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