From 7e7e94e33655d400671723dd4dda92b9092dfca7 Mon Sep 17 00:00:00 2001 From: Niels Vanpachtenbeke <10651054+Nielsvanpach@users.noreply.github.com> Date: Mon, 18 Dec 2023 09:09:18 +0100 Subject: [PATCH 1/4] minimize amount of ci runs on tests + add time-out --- .github/workflows/run-tests-pest.yml | 10 +++++++++- .github/workflows/run-tests-phpunit.yml | 10 +++++++++- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/.github/workflows/run-tests-pest.yml b/.github/workflows/run-tests-pest.yml index bac4eb8..82df4aa 100644 --- a/.github/workflows/run-tests-pest.yml +++ b/.github/workflows/run-tests-pest.yml @@ -1,10 +1,18 @@ name: Tests -on: [push, pull_request] +on: + push: + paths: + - '**.php' + - '.github/workflows/run-tests-pest.yml' + - 'phpunit.xml.dist' + - 'composer.json' + - 'composer.lock' jobs: test: runs-on: ${{ matrix.os }} + timeout-minutes: 5 strategy: fail-fast: true matrix: diff --git a/.github/workflows/run-tests-phpunit.yml b/.github/workflows/run-tests-phpunit.yml index 6529be6..6b14bca 100644 --- a/.github/workflows/run-tests-phpunit.yml +++ b/.github/workflows/run-tests-phpunit.yml @@ -1,10 +1,18 @@ name: Tests -on: [push, pull_request] +on: + push: + paths: + - '**.php' + - '.github/workflows/run-tests-pest.yml' + - 'phpunit.xml.dist' + - 'composer.json' + - 'composer.lock' jobs: test: runs-on: ${{ matrix.os }} + timeout-minutes: 5 strategy: fail-fast: true matrix: From 6bfb14cb2211656b9696a2c4cfd07edca12059fc Mon Sep 17 00:00:00 2001 From: Niels Vanpachtenbeke <10651054+Nielsvanpach@users.noreply.github.com> Date: Mon, 18 Dec 2023 09:09:50 +0100 Subject: [PATCH 2/4] add timeout to all flows --- .github/workflows/dependabot-auto-merge.yml | 5 +++-- .github/workflows/fix-php-code-style-issues-cs-fixer.yml | 1 + .github/workflows/fix-php-code-style-issues-pint.yml | 1 + .github/workflows/update-changelog.yml | 1 + 4 files changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/dependabot-auto-merge.yml b/.github/workflows/dependabot-auto-merge.yml index b34f094..1c6bb30 100644 --- a/.github/workflows/dependabot-auto-merge.yml +++ b/.github/workflows/dependabot-auto-merge.yml @@ -8,15 +8,16 @@ permissions: jobs: dependabot: runs-on: ubuntu-latest + timeout-minutes: 5 if: ${{ github.actor == 'dependabot[bot]' }} steps: - + - name: Dependabot metadata id: metadata uses: dependabot/fetch-metadata@v1.6.0 with: github-token: "${{ secrets.GITHUB_TOKEN }}" - + - name: Auto-merge Dependabot PRs for semver-minor/patch updates if: ${{steps.metadata.outputs.update-type != 'version-update:semver-major'}} run: gh pr merge --auto --merge "$PR_URL" diff --git a/.github/workflows/fix-php-code-style-issues-cs-fixer.yml b/.github/workflows/fix-php-code-style-issues-cs-fixer.yml index 3e044ea..5495e26 100644 --- a/.github/workflows/fix-php-code-style-issues-cs-fixer.yml +++ b/.github/workflows/fix-php-code-style-issues-cs-fixer.yml @@ -8,6 +8,7 @@ permissions: jobs: php-cs-fixer: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - name: Checkout code diff --git a/.github/workflows/fix-php-code-style-issues-pint.yml b/.github/workflows/fix-php-code-style-issues-pint.yml index a5b1b60..4a9038f 100644 --- a/.github/workflows/fix-php-code-style-issues-pint.yml +++ b/.github/workflows/fix-php-code-style-issues-pint.yml @@ -11,6 +11,7 @@ permissions: jobs: php-code-styling: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - name: Checkout code diff --git a/.github/workflows/update-changelog.yml b/.github/workflows/update-changelog.yml index a817f81..0390685 100644 --- a/.github/workflows/update-changelog.yml +++ b/.github/workflows/update-changelog.yml @@ -10,6 +10,7 @@ permissions: jobs: update: runs-on: ubuntu-latest + timeout-minutes: 5 steps: - name: Checkout code From ae039291c56fec20c853c165366eaa075b1915de Mon Sep 17 00:00:00 2001 From: Niels Vanpachtenbeke <10651054+Nielsvanpach@users.noreply.github.com> Date: Mon, 18 Dec 2023 09:12:22 +0100 Subject: [PATCH 3/4] run code fixer workflow only on .php changes --- .github/workflows/fix-php-code-style-issues-cs-fixer.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/fix-php-code-style-issues-cs-fixer.yml b/.github/workflows/fix-php-code-style-issues-cs-fixer.yml index 5495e26..28d0c02 100644 --- a/.github/workflows/fix-php-code-style-issues-cs-fixer.yml +++ b/.github/workflows/fix-php-code-style-issues-cs-fixer.yml @@ -1,6 +1,9 @@ name: Check & fix styling -on: [push] +on: + push: + paths: + - '**.php' permissions: contents: write From 6dd7bb69cc66b1b4d77e7ec99dbaa5732a7a452a Mon Sep 17 00:00:00 2001 From: Niels Vanpachtenbeke <10651054+Nielsvanpach@users.noreply.github.com> Date: Mon, 18 Dec 2023 09:13:33 +0100 Subject: [PATCH 4/4] fix typo --- .github/workflows/run-tests-phpunit.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/run-tests-phpunit.yml b/.github/workflows/run-tests-phpunit.yml index 6b14bca..b093bc8 100644 --- a/.github/workflows/run-tests-phpunit.yml +++ b/.github/workflows/run-tests-phpunit.yml @@ -4,7 +4,7 @@ on: push: paths: - '**.php' - - '.github/workflows/run-tests-pest.yml' + - '.github/workflows/run-tests-phpunit.yml' - 'phpunit.xml.dist' - 'composer.json' - 'composer.lock'