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

[xi...@apache.org]

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

xiaoxiang 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 d7a5313  Revert "CI: use 'git diff $commits' as a whole patchset to do checkpatch"
d7a5313 is described below

commit d7a5313acac4a3cbac441da67f8315b525cee853
Author: liuhaitao <li...@xiaomi.com>
AuthorDate: Sat May 9 13:13:12 2020 +0800

    Revert "CI: use 'git diff $commits' as a whole patchset to do checkpatch"
    
    This reverts commit 9c4cdc5331c1989b9a82e392255e516b16a5a47b.
    
    If one PR is on a former master code base, using 'git diff $commits' would result in
    abnormal checkpatch report sometimes. So revert it anyway.
---
 .github/workflows/check.yml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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