This reverts commit 350131d00e.
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.
Something change on the GitHub side that has broken credentials
between the the fetch and buld workflow steps. There is no
way to tell the checkout action to only re-auth so we have it
checkout the repo again, but it will use what it already has.
In the future we can change fetch step to just compute the
target refs and store them in a script that we can call to
set the environment variables for the targets much like
this patch.
Signed-off-by: Brennan Ashton <bashton@brennanashton.com>
So far, the use of the new pull request template has been disastrous. People are ignoring the writing instructions the template is polluting the PR history. This change just removes the boilerplate and writing instructions fromt he template. We don't need to see this unmodified serveral times per day.
There is chance pull docker container failure in build jobs,
remove the additional pull docker container step may make it
more stable. Meanwhile, it also saves some time for build job.
Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
As discussion in https://github.com/apache/incubator-nuttx/pull/549 shows,
update github action workflow to let build jobs do not depend on nxstyle
check job in order to make sure each PR (including some nxstyle waiving cases)
be built at least.
Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
Github action CI workflow steps as below:
1. Use docker container with build essential tools preinstalled
2. nxstyle check pull request with checkpatch.sh
3. Call testing cibuild.sh to do jobs matrix check builds
Signed-off-by: liuhaitao <liuhaitao@xiaomi.com>
Brennan Ashton