Test images before pushing them (#114)

* Test images before pushing them * Wording * Only trigger CI when we change a layer * Blackfire fix * syntax
brefphp · Nov 27, 2020 · 3ef1ff7 · 3ef1ff7
1 parent 9ab717f
commit 3ef1ff7
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 9 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,10 +1,9 @@
 name: CI
 
 on:
-  pull_request: ~
-  push:
-    branches:
-      - master
+  pull_request:
+    paths:
+      - layers/**
 
 jobs:
   build:
@@ -44,8 +43,8 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v2
 
-      - name: Build layer
+      - name: Build docker images
         run: layer=${{ matrix.layer }} make docker-images
 
-      - name: Test layer
-        run:  layer=${{ matrix.layer }} make test
+      - name: Test images
+        run: layer=${{ matrix.layer }} make test
diff --git a/.github/workflows/deploy-docker.yml b/.github/workflows/deploy-docker.yml
@@ -30,6 +30,9 @@ jobs:
       - name: Build images
         run: make docker-images
 
+      - name: Test images
+        run: make test
+
       - name: Publish
         env:
           DOCKER_TAG: ${{ steps.get_version.outputs.VERSION }}

diff --git a/layers/blackfire/versions.json b/layers/blackfire/versions.json
@@ -1,5 +1,4 @@
 {
-    "72": "1.44.0",
     "73": "1.46.0",
     "74": "1.46.0"
-}
+}