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