You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2020/05/09 06:10:45 UTC

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

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.git


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

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

    Revert "CI: use 'git diff $commits' as a whole patchset to do checkpatch"
    
    This reverts commit 350131d00e1f0dddcd9252ec6a535830b08f0d1c.
    
    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 21fc942..242496a 100644
--- a/.github/workflows/check.yml
+++ b/.github/workflows/check.yml
@@ -32,4 +32,5 @@ jobs:
         cd nuttx
         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