Merge pull request #62 from timirey/hotfix/remove-is-connected-method-from-sc

Update workflows

Author: timirey

.github/workflows/pull-request.yml

@@ -1,40 +1,40 @@
 name: pull request
 
 on:
-  pull_request:
-    types:
-      - opened
-      - synchronize
-      - reopened
-    branches:
-      - dev
-      - main
+    pull_request:
+        types:
+            - opened
+            - synchronize
+            - reopened
+        branches:
+            - dev
+            - main
 
 jobs:
-  check-pull-request:
-    runs-on: ubuntu-latest
+    check-pull-request:
+        runs-on: ubuntu-latest
 
-    steps:
-      - name: Check branch prefix
-        run: |
-          BRANCH_NAME=${GITHUB_HEAD_REF}
-          TARGET_BRANCH=${GITHUB_BASE_REF}
+        steps:
+            -   name: Check branch prefix
+                run: |
+                    BRANCH_NAME=${GITHUB_HEAD_REF}
+                    TARGET_BRANCH=${GITHUB_BASE_REF}
 
-          if [[ "$TARGET_BRANCH" == "dev" ]]; then
-            if [[ "$BRANCH_NAME" =~ ^feature/|^release/|^hotfix/ ]]; then
-              echo "Branch name is valid for dev."
-            else
-              echo "Error: Branch name for dev must start with 'feature/', 'release/', or 'hotfix/'."
-              exit 1
-            fi
-          elif [[ "$TARGET_BRANCH" == "main" ]]; then
-            if [[ "$BRANCH_NAME" =~ ^release/|^hotfix/ ]]; then
-              echo "Branch name is valid for main."
-            else
-              echo "Error: Branch name for main must start with 'release/' or 'hotfix/'."
-              exit 1
-            fi
-          else
-            echo "Error: Unsupported target branch."
-            exit 1
-          fi
+                    if [[ "$TARGET_BRANCH" == "dev" ]]; then
+                      if [[ "$BRANCH_NAME" =~ ^feature/|^release/|^hotfix/ ]]; then
+                        echo "Branch name is valid for dev."
+                      else
+                        echo "Error: Branch name for dev must start with 'feature/', 'release/', or 'hotfix/'."
+                        exit 1
+                      fi
+                    elif [[ "$TARGET_BRANCH" == "main" ]]; then
+                      if [[ "$BRANCH_NAME" =~ ^release/|^hotfix/ ]]; then
+                        echo "Branch name is valid for main."
+                      else
+                        echo "Error: Branch name for main must start with 'release/' or 'hotfix/'."
+                        exit 1
+                      fi
+                    else
+                      echo "Error: Unsupported target branch."
+                      exit 1
+                    fi

.github/workflows/tests.yml

@@ -1,32 +1,40 @@
 name: tests
 
 on:
-  push:
-    branches:
-      - main
-      - dev
+    push:
+        branches:
+            - main
+            - dev
+    pull_request:
+        types:
+            - opened
+            - synchronize
+            - reopened
+        branches:
+            - dev
+            - main
 
 jobs:
-  check-tests:
-    runs-on: ubuntu-latest
+    check-tests:
+        runs-on: ubuntu-latest
 
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
+        steps:
+            -   name: Checkout code
+                uses: actions/checkout@v4
 
-      - name: Setup PHP
-        uses: shivammathur/setup-php@v2
-        with:
-          php-version: 8.3
+            -   name: Setup PHP
+                uses: shivammathur/setup-php@v2
+                with:
+                    php-version: 8.3
 
-      - name: Install Dependencies
-        run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+            -   name: Install Dependencies
+                run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
 
-      - name: Run PHP CodeSniffer
-        run: vendor/bin/phpcs --severity=6
+            -   name: Run PHP CodeSniffer
+                run: vendor/bin/phpcs --severity=6
 
-      - name: Run PHP Stan
-        run: vendor/bin/phpstan analyse src --level=8
+            -   name: Run PHP Stan
+                run: vendor/bin/phpstan analyse src --level=8
 
-      - name: Execute tests (Unit and Feature tests) via PHPUnit/Pest
-        run: vendor/bin/pest
+            -   name: Execute tests (Unit and Feature tests) via PHPUnit/Pest
+                run: vendor/bin/pest

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## [6.0.3](https://github.com/timirey/xapi-php/compare/6.0.2..6.0.3) - 2024-07-30
 
 - Remove `isConnected()` from `SocketConnection::class`.
+- Updated GitHub workflows for better checks.
 
 ## [6.0.2](https://github.com/timirey/xapi-php/compare/6.0.1..6.0.2) - 2024-07-30