On 22/03/2022 20.14, Yonggang Luo wrote:
Signed-off-by: Yonggang Luo <luoyongg...@gmail.com>

Missing patch description ... why is this needed? I can't see anything related to Cirrus-CI in here?

---
  .github/workflows/main.yml | 29 +++++++++++++++++++++++++++++
  MAINTAINERS                |  4 +++-
  2 files changed, 32 insertions(+), 1 deletion(-)
  create mode 100644 .github/workflows/main.yml

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
new file mode 100644
index 0000000000..85a392b962
--- /dev/null
+++ b/.github/workflows/main.yml
@@ -0,0 +1,29 @@
+# This is a basic workflow to help you get started with Actions
+
+name: CI
+
+# Controls when the action will run. Triggers the workflow on push or pull 
request
+# events but only for the master branch
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+# A workflow run is made up of one or more jobs that can run sequentially or 
in parallel
+jobs:
+  check-patch:
+    name: Check Patch
+    runs-on: ubuntu-latest
+    steps:
+      # Checks-out your repository under $GITHUB_WORKSPACE, so your job can 
access it
+      - uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+      - name: Install dependent packages
+        run: sudo apt install perl
+      - name: Checkpatch
+        run: |
+          git fetch https://git.qemu.org/git/qemu.git master
+          git tag base_tag_to_rebase FETCH_HEAD --force
+          ./scripts/checkpatch.pl --mailback base_tag_to_rebase..
diff --git a/MAINTAINERS b/MAINTAINERS
index cc364afef7..6d1dcc36b0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3612,10 +3612,12 @@ Build and test automation, general continuous 
integration
  M: Alex Bennée <alex.ben...@linaro.org>
  M: Philippe Mathieu-Daudé <f4...@amsat.org>
  M: Thomas Huth <th...@redhat.com>
+M: Yonggang Luo <luoyonggang@gmail>
  R: Wainer dos Santos Moschetta <waine...@redhat.com>
  R: Beraldo Leal <bl...@redhat.com>
  S: Maintained
-F: .github/lockdown.yml
+F: .github/workflows/lockdown.yml
+F: .github/workflows/main.yml
  F: .gitlab-ci.yml
  F: .gitlab-ci.d/
  F: .travis.yml

You also don't mention that change anywhere. And IIRC you've also only been involved in the Windows-related CI tasks so far? So it might make more sense to update the "Windows Hosted Continuous Integration" section instead?

 Thomas


Reply via email to