[tracker/wip/carlosg/code-style-checks] bar




commit 259b8f3cbbb0a41b742c857df5210b4e2980ef46
Author: Carlos Garnacho <carlosg gnome org>
Date:   Mon Jun 14 01:21:43 2021 +0200

    bar

 .gitlab-ci.yml | 8 ++++++--
 check-style.py | 3 +++
 2 files changed, 9 insertions(+), 2 deletions(-)
---
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index e0a63c3ef..9d289b90c 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -181,12 +181,16 @@ check-code-style:
     - build-fedora-container@x86_64
   stage: code-review
   script:
-    - export common_parent_sha=$(diff --old-line-format='' --new-line-format='' <(git rev-list 
--first-parent "origin/${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}") <(git rev-list 
--first-parent HEAD) | head -1) ;
+    - git fetch origin ;
+      export common_parent_sha=$(diff --old-line-format='' --new-line-format='' <(git rev-list 
--first-parent "origin/${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}") <(git rev-list 
--first-parent HEAD) | head -1) ;
+      echo $common_parent_sha ;
       ./check-style.py --sha $common_parent_sha ;
       if [ $? -ne 0 ] ;
       then
         echo -e "Issue the following command in your local tree to apply the suggested changes (needs 
uncrustify installed):\n\n $ git rebase "origin/${CI_MERGE_REQUEST_TARGET_BRANCH_NAME:-${CI_DEFAULT_BRANCH}}" 
--exec \"./check-style.py -r\" \n" ;
-      fi
+      else
+        echo -e "All looks good.\n" ;
+      fi;
   allow_failure: true
 
 .build-template: &build
diff --git a/check-style.py b/check-style.py
index 9ce886117..1a1e79d9a 100755
--- a/check-style.py
+++ b/check-style.py
@@ -110,8 +110,11 @@ args = parser.parse_args()
 sha = args.sha or 'HEAD^'
 rewrite = args.rewrite
 
+print ("Run diff")
 diff = run_diff(sha)
+print ("Find chunks")
 chunks = find_chunks(diff)
+print ("Reformat chunks")
 changed = reformat_chunks(chunks, rewrite)
 
 if rewrite is True:


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]