(yetus) branch main updated: YETUS-1277. Skip per-sub-check commit statuses under GitHub Actions (#386)

[ndimiduk]

This is an automated email from the ASF dual-hosted git repository.

ndimiduk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/yetus.git


The following commit(s) were added to refs/heads/main by this push:
     new d3ec18c3 YETUS-1277. Skip per-sub-check commit statuses under GitHub 
Actions (#386)
d3ec18c3 is described below

commit d3ec18c3999f6f12812ff65d9ffef7a4bcf0262a
Author: Nick Dimiduk <ndimi...@apache.org>
AuthorDate: Thu May 21 10:15:59 2026 +0200

    YETUS-1277. Skip per-sub-check commit statuses under GitHub Actions (#386)
    
    Under GHA, Yetus encodes result severity (error:/warning:) in the GitHub
    commit status context string. Since GitHub keys statuses by context, a
    re-run that changes a check's outcome never overwrites the prior status.
    The per-sub-check detail is already available in the PR comment and the
    GHA step summary, so commit statuses are redundant under GHA.
---
 precommit/src/main/shell/plugins.d/github.sh | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/precommit/src/main/shell/plugins.d/github.sh 
b/precommit/src/main/shell/plugins.d/github.sh
index c5613cec..cef6bab1 100755
--- a/precommit/src/main/shell/plugins.d/github.sh
+++ b/precommit/src/main/shell/plugins.d/github.sh
@@ -1223,6 +1223,13 @@ function github_finalreport
 
   github_end_checkrun "${result}"
 
+  # Under GHA the per-sub-check detail is already in the PR comment and the
+  # step summary. Commit statuses encode severity in the context key, so a
+  # re-run that changes outcome never overwrites the old status (YETUS-1277).
+  if [[ "${ROBOTTYPE}" == 'githubactions' ]]; then
+    return 0
+  fi
+
   if [[ "${ROBOTTYPE}" ]]; then
     header="Apache Yetus(${ROBOTTYPE})"
   else