[yetus] branch main updated: YETUS-1007. Logging when adding GitHub commit statuses (#139)

[aajisaka]

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

aajisaka 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 b103002  YETUS-1007. Logging when adding GitHub commit statuses (#139)
b103002 is described below

commit b103002eef0d7dcc9cab3e469f386fd6308050a6
Author: Akira Ajisaka <aajis...@apache.org>
AuthorDate: Tue Sep 29 23:39:37 2020 +0900

    YETUS-1007. Logging when adding GitHub commit statuses (#139)
    
    Signed-off-by: Allen Wittenauer <a...@apache.org>
---
 precommit/src/main/shell/test-patch.d/github.sh | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/precommit/src/main/shell/test-patch.d/github.sh 
b/precommit/src/main/shell/test-patch.d/github.sh
index d303829..20325aa 100755
--- a/precommit/src/main/shell/test-patch.d/github.sh
+++ b/precommit/src/main/shell/test-patch.d/github.sh
@@ -704,6 +704,7 @@ function github_status_write()
 {
   declare filename=$1
   declare -a githubauth
+  declare retval=0
 
   if [[ -n "${GITHUB_TOKEN}" ]]; then
     githubauth=(-H "Authorization: token ${GITHUB_TOKEN}")
@@ -723,6 +724,12 @@ function github_status_write()
     --location \
     "${GITHUB_API_URL}/repos/${GITHUB_REPO}/statuses/${GITHUB_SHA}" \
     > /dev/null
+
+  retval=$?
+  if [[ ${retval} -gt 0 ]]; then
+    echo "githubstatus-report: Failed to write status. Maybe the credential 
does not have write access to repo:status."
+  fi
+  return ${retval}
 }
 
 ## @description  Print out the finished details to the Github PR