Update OCI workflow of server

* Only on pull requests
* Concurrency group
* Bump used actions
* Add summary

Signed-off-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>

diff --git a/.github/workflows/oci.yml b/.github/workflows/oci.yml
index 8d09578cf4e2e4a50a5908d0c1296ec53fa18577..a7e9417e4e6ebc41f5f8bcc8949e19832517e89e 100644 (file)
--- a/.github/workflows/oci.yml
+++ b/.github/workflows/oci.yml
@@ -1,46 +1,39 @@
 name: PHPUnit
 
-on:
-  pull_request:
-  push:
-    branches:
-      - master
-      - stable*
+on: pull_request
+
+permissions:
+  contents: read
+
+concurrency:
+  group: phpunit-oci-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
 
 jobs:
-  phpunit-oci8:
-    runs-on: ubuntu-20.04
+  phpunit-oci:
+    runs-on: ubuntu-latest
 
     strategy:
-      fail-fast: false
       matrix:
         php-versions: [ '7.4', '8.0', '8.1']
-        databases: [ 'oci' ]
-
-    name: php${{ matrix.php-versions }}-${{ matrix.databases }}
 
     services:
       oracle:
-        image: deepdiver/docker-oracle-xe-11g # "wnameless/oracle-xe-11g-r2"
+        image: deepdiver/docker-oracle-xe-11g # 'wnameless/oracle-xe-11g-r2'
         ports:
-          - "1521:1521"
+          - 1521:1521/tcp
 
     steps:
       - name: Checkout server
-        uses: actions/checkout@v2
-
-      - name: Checkout submodules
-        shell: bash
-        run: |
-          auth_header="$(git config --local --get http.https://github.com/.extraheader)"
-          git submodule sync --recursive
-          git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1
+        uses: actions/checkout@v3
+        with:
+          submodules: true
 
       - name: Set up php ${{ matrix.php-versions }}
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ matrix.php-versions }}
-          extensions: ctype,curl,dom,fileinfo,gd,imagick,intl,json,mbstring,oci8,openssl,pdo_sqlite,posix,sqlite,xml,zip
+          extensions: ctype, curl, dom, fileinfo, gd, imagick, intl, json, mbstring, oci8, openssl, pdo_sqlite, posix, sqlite, xml, zip
           tools: phpunit:9
           coverage: none
 
@@ -53,3 +46,17 @@ jobs:
       - name: PHPUnit
         working-directory: tests
         run: phpunit --configuration phpunit-autotest.xml --group DB,SLOWDB
+
+  summary:
+    permissions:
+      contents: none
+    runs-on: ubuntu-latest
+    needs: phpunit-oci
+
+    if: always()
+
+    name: phpunit-oci-summary
+
+    steps:
+      - name: Summary status
+        run: if ${{ needs.phpunit-oci.result != 'success' }}; then exit 1; fi