This is an automated email from the ASF dual-hosted git repository.
gnodet pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/main by this push:
new 05d593bc0dfe CAMEL-23451: skip non-module files in incremental build
instead of aborting
05d593bc0dfe is described below
commit 05d593bc0dfe33a91d6b02285d58b09b3a717de1
Author: Guillaume Nodet <[email protected]>
AuthorDate: Mon May 11 11:25:28 2026 +0200
CAMEL-23451: skip non-module files in incremental build instead of aborting
Regression from #22247: when the CI rationalization removed the buildAll
mode,
the projectRoot == "." case was changed from "build everything" to exit 0
(skip
all tests). Files in directories without a pom.xml (e.g. proposals/*.adoc,
README.md) caused findProjectRoot to return ".", aborting all test execution
even when the PR also contained real code changes.
Fix: skip non-module files with continue instead of aborting. Root-level POM
changes are already handled by the grep and Scalpel dependency analysis.
---
.github/actions/incremental-build/incremental-build.sh | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/.github/actions/incremental-build/incremental-build.sh
b/.github/actions/incremental-build/incremental-build.sh
index f0bce642149f..a1efc1e4c960 100755
--- a/.github/actions/incremental-build/incremental-build.sh
+++ b/.github/actions/incremental-build/incremental-build.sh
@@ -512,10 +512,8 @@ main() {
local projectRoot
projectRoot=$(findProjectRoot "${project}")
if [[ ${projectRoot} = "." ]]; then
- echo "The root project is affected, skipping targeted module testing"
- echo "<!-- ci-tested-modules -->" > "incremental-test-comment.md"
- echo ":information_source: CI did not run targeted module tests (root
project files changed)." >> "incremental-test-comment.md"
- exit 0
+ echo " Skipping non-module file: ${project} (no parent pom.xml found)"
+ continue
elif [[ ${projectRoot} != "${lastProjectRoot}" ]]; then
totalAffected=$((totalAffected + 1))
pl="$pl,${projectRoot}"