davsclaus commented on code in PR #28885:
URL: https://github.com/apache/camel/pull/28885#discussion_r4444028237
##########
.github/workflows/camel-launcher-native-exe.yml:
##########
@@ -224,10 +224,13 @@ jobs:
# Plain mvn: once -am is gone this reactor is only a few modules, so
the mvnd
# daemon brings no parallelism benefit and adds a cold-start failure
mode.
# -P apache-snapshots is required here so the upstream modules resolve.
+ #
+ # With the tests: this is the only build with the native exe files,
which the tests that need
+ # -Dcamel.exe.build=true check (PackagePlanTest, and the *IT of the
distributions in verify).
run: |
Review Comment:
Right, `PackagePlanTest` only names `-Dcamel.exe.build=true` in a Javadoc;
only the `*IT` are gated on it. Fixed the comment in ae2a9d839d47.
_Claude Code on behalf of Claus Ibsen_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
