Check runs are now also trigged by PRs

This should catch issues before they are merged.
Relates to #134

.github/workflows/checks.yml

@@ -1,6 +1,6 @@
 name: Tests and Checks
 
-on: [push]
+on: [push, pull_request]
 
 jobs:
   checks:
@@ -11,6 +11,8 @@ jobs:
         os: ["macos-latest", "ubuntu-latest"]
     steps:
       - uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }} # Checkout pull request HEAD commit instead of merge commit
 
       - name: Setup python
         uses: actions/setup-python@v1
@@ -58,7 +60,7 @@ jobs:
       #     github-token: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Gitlint check
-        run: ./run_tests.sh -g
+        run: ./run_tests.sh -g --debug
 
   windows-checks:
     runs-on: windows-latest
@@ -67,6 +69,8 @@ jobs:
         python-version: [2.7, 3.5]
     steps:
       - uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }} # Checkout pull request HEAD commit instead of merge commit
 
       - name: Setup python
         uses: actions/setup-python@v1

run_tests.sh

@@ -121,7 +121,7 @@ run_integration_tests(){
 
 run_git_check(){
     echo -ne "Running gitlint...${RED}"
-    RESULT=$(gitlint 2>&1)
+    RESULT=$(gitlint $testargs 2>&1)
     local exit_code=$?
     handle_test_result $exit_code "$RESULT"
     # FUTURE: check if we use str() function: egrep -nriI "( |\(|\[)+str\(" gitlint | egrep -v "\w*#(.*)"