You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by ag...@apache.org on 2020/05/08 16:26:26 UTC

[incubator-nuttx-apps] branch master updated: CI: use 'git diff $commits' as a whole patchset to do checkpatch

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

aguettouche pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git


The following commit(s) were added to refs/heads/master by this push:
     new 9c4cdc5  CI: use 'git diff $commits' as a whole patchset to do checkpatch
9c4cdc5 is described below

commit 9c4cdc5331c1989b9a82e392255e516b16a5a47b
Author: liuhaitao <li...@xiaomi.com>
AuthorDate: Fri May 8 11:23:50 2020 +0800

    CI: use 'git diff $commits' as a whole patchset to do checkpatch
    
    So avoid the duplicate print logs sometimes and rename file not opened issue.
    
    Signed-off-by: liuhaitao <li...@xiaomi.com>
---
 .github/workflows/check.yml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/check.yml b/.github/workflows/check.yml
index ac3c144..2c0d586 100644
--- a/.github/workflows/check.yml
+++ b/.github/workflows/check.yml
@@ -38,5 +38,4 @@ jobs:
         cd apps
         commits=`git log -1 --merges --pretty=format:%P | awk -F" " '{ print $1 ".." $2 }'`
         git log --oneline $commits
-        echo "../nuttx/tools/checkpatch.sh -g $commits"
-        ../nuttx/tools/checkpatch.sh -g $commits
+        git diff $commits | ../nuttx/tools/checkpatch.sh -