From: Paul Spooren <redacted>
Date: Fri, 5 Mar 2021 09:16:00 +0000 (-1000)
Subject: CI: checkout HEAD commit rather than merge commit
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=13c1f2bcda33ab8fc17ede1f43f60e0aac8b7cab;p=openwrt-packages.git

CI: checkout HEAD commit rather than merge commit

GitHub CI actions/checkout uses a merge commit which isn't compatible
with our formality checks. Instead checkout the pull request HEAD.

Signed-off-by: Paul Spooren <redacted>
---

diff --git a/.github/workflows/formal.yml b/.github/workflows/formal.yml
index 7a7a6b0aa..25609174e 100644
--- a/.github/workflows/formal.yml
+++ b/.github/workflows/formal.yml
@@ -13,6 +13,7 @@ jobs:
     steps:
       - uses: actions/checkout@v2
         with:
+          ref: ${{ github.event.pull_request.head.sha }}
           fetch-depth: 0
 
       - name: Determine branch name
@@ -23,9 +24,6 @@ jobs:
 
       - name: Test formalities
         run: |
-          # remove GitHubs merge commit
-          git rebase "origin/$BRANCH"
-
           source .github/workflows/ci_helpers.sh
 
           RET=0