fix(php-idna and php-sdk workflows): reviewed coverage report generat…

…ion implementation
centralnicgroup-opensource · Mar 22, 2024 · 073b910 · 073b910
1 parent df40c79
commit 073b910
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 8 deletions.
diff --git a/.github/workflows/php-idna-translator-release.yml b/.github/workflows/php-idna-translator-release.yml
@@ -33,14 +33,19 @@ jobs:
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ vars.RTLDEV_MW_CI_PHP_VERSION }}
+          coverage: xdebug
       - name: Install dependencies
         run: |
           npm ci
           echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
           composer update --no-dev
+      - name: PHP Class Documentation
+        run: composer run-script docs
+      - name: Code Coverage Report
+        run: composer run-script coverage
       - name: Release
         env:
           GITHUB_TOKEN: ${{ secrets.RTLDEV_MW_CI_TOKEN }}
           TEAMS_NOTIFICATION_URI: ${{ secrets.RTLDEV_MW_CI_NOTIFICATION_URI }}
           COMMIT_SHA: ${{ github.sha }}
-        run: npx semantic-release
+        run: npx semantic-release
diff --git a/.github/workflows/php-idna-translator-test.yml b/.github/workflows/php-idna-translator-test.yml
@@ -14,10 +14,6 @@ jobs:
         with:
           fetch-depth: 0
           persist-credentials: false
-      - name: Install Dependencies::PHP${{ matrix.php-version }}
-        run: |
-          echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
-          composer update          
       - name: Super Linter Code Base
         uses: github/super-linter/slim@v5
         env:
@@ -44,9 +40,14 @@ jobs:
         with:
           fetch-depth: 0
           persist-credentials: false
+      - name: Install PHP ${{ matrix.php-version }}
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+          coverage: xdebug          
       - name: Install Dependencies::PHP${{ matrix.php-version }}
         run: |
           echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
           composer update
-      - name: Code Validation
-        run: composer run-script test
+      - name: Code Validation & Coverage
+        run: composer run-script coverage
diff --git a/.github/workflows/php-sdk-release.yml b/.github/workflows/php-sdk-release.yml
@@ -33,12 +33,16 @@ jobs:
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ vars.RTLDEV_MW_CI_PHP_VERSION }}
+          coverage: xdebug
       - name: Install dependencies
         run: |
           npm ci
           echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"
           composer update --no-dev
-          composer run-script docs
+      - name: PHP Class Documentation
+        run: composer run-script docs
+      - name: Code Coverage Report
+        run: composer run-script test
       - name: Release
         env:
           GITHUB_TOKEN: ${{ secrets.RTLDEV_MW_CI_TOKEN }}

diff --git a/.github/workflows/php-sdk-test.yml b/.github/workflows/php-sdk-test.yml
@@ -41,6 +41,11 @@ jobs:
         with:
           fetch-depth: 0
           persist-credentials: false
+      - name: Install PHP ${{ matrix.php-version }}
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+          coverage: xdebug
       - name: Install Dependencies::PHP${{ matrix.php-version }}
         run: |
           echo "$HOME/.composer/vendor/bin" >> "$GITHUB_PATH"