Run GitHub actions on pull requests

quick-lint · Oct 13, 2020 · cadbfbb · cadbfbb
1 parent c242fc8
commit cadbfbb
Show file tree

Hide file tree

Showing 5 changed files with 62 additions and 14 deletions.
diff --git a/.github/workflows/build-and-test-web-demo.yml b/.github/workflows/build-and-test-web-demo.yml
@@ -15,7 +15,10 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 name: build and test web demo
-on: [push]
+on:
+  push:
+  pull_request:
+    types: [opened, synchronize]
 
 jobs:
   build:
@@ -25,8 +28,14 @@ jobs:
       - uses: mymindstorm/setup-emsdk@v6
         with:
           version: 2.0.4
-      - name: check out
-        uses: actions/checkout@v1
+      - name: checkout
+        if: ${{ github.event_name == 'push' }}
+        uses: actions/checkout@v2
+      - name: checkout
+        if: ${{ github.event_name == 'pull_request' }}
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
 
       - name: C++ configure
         run: emcmake cmake -S . -B build -DCMAKE_BUILD_TYPE=Release

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -15,7 +15,10 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 name: build and test
-on: [push]
+on:
+  push:
+  pull_request:
+    types: [opened, synchronize]
 
 jobs:
   build:
@@ -56,8 +59,15 @@ jobs:
         ) }}
 
     steps:
-      - name: check out
-        uses: actions/checkout@v1
+      - name: checkout
+        if: ${{ github.event_name == 'push' }}
+        uses: actions/checkout@v2
+      - name: checkout
+        if: ${{ github.event_name == 'pull_request' }}
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
       - name: configure
         run: |
           env | grep '^CMAKE\|^QUICK_LINT_JS' | sort

diff --git a/.github/workflows/check-format.yml b/.github/workflows/check-format.yml
@@ -15,14 +15,23 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 name: check format
-on: [push]
+on:
+  push:
+  pull_request:
+    types: [opened, synchronize]
 
 jobs:
   check_format:
     name: check format
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      - if: ${{ github.event_name == 'push' }}
+        uses: actions/checkout@v2
+      - if: ${{ github.event_name == 'pull_request' }}
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
       - run: clang-format --version
       - run: ./tools/format
       - run: git diff --exit-code
diff --git a/.github/workflows/simple-build.yml b/.github/workflows/simple-build.yml
@@ -15,7 +15,10 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 name: simple build
-on: [push]
+on:
+  push:
+  pull_request:
+    types: [opened, synchronize]
 
 jobs:
   build:
@@ -34,7 +37,14 @@ jobs:
       CXXFLAGS: ${{ matrix.toolchain.CXXFLAGS }}
 
     steps:
-      - name: check out
-        uses: actions/checkout@v1
+      - name: checkout
+        if: ${{ github.event_name == 'push' }}
+        uses: actions/checkout@v2
+      - name: checkout
+        if: ${{ github.event_name == 'pull_request' }}
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
       - name: build
         run: '"${CXX}" ${CXXFLAGS} -o quick-lint-js -I src -I vendor/boost src/*.cpp vendor/boost/libs/container/src/*.cpp vendor/boost/libs/container/src/alloc_lib.c'
diff --git a/.github/workflows/test-install.yml b/.github/workflows/test-install.yml
@@ -15,15 +15,25 @@
 # along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
 name: test install
-on: [push]
+on:
+  push:
+  pull_request:
+    types: [opened, synchronize]
 
 jobs:
   build:
     name: test install
     runs-on: ubuntu-latest
     steps:
-      - name: check out
-        uses: actions/checkout@v1
+      - name: checkout
+        if: ${{ github.event_name == 'push' }}
+        uses: actions/checkout@v2
+      - name: checkout
+        if: ${{ github.event_name == 'pull_request' }}
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
       - name: configure
         run: cmake -DCMAKE_C_COMPILER=gcc-8 -DCMAKE_CXX_COMPILER=g++-8 -DBUILD_TESTING=OFF -S . -B .
       - name: build