From 54b95b32784d29f5e1d6385924a02ab2e7667971 Mon Sep 17 00:00:00 2001 From: MitchBred Date: Thu, 6 Aug 2026 14:09:45 +0200 Subject: [PATCH 1/2] fix: scope het lint-script tot de eigen bronbestanden Kale phplint scant ook vendor en struikelt dan over de fixture met een opzettelijke syntaxfout in overtrue/phplint zelf. Nu expliciet src, tests en rector.php, wat op dezelfde 24 bestanden uitkomt als de GitHub Action rapporteerde. Co-Authored-By: Claude Opus 5 (1M context) --- composer.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/composer.json b/composer.json index d13b09c4..4c3228e3 100644 --- a/composer.json +++ b/composer.json @@ -52,6 +52,6 @@ }, "scripts": { "test": "phpunit", - "lint": "phplint" + "lint": "phplint src tests rector.php" } } From 60803b90d299fb324922fb7a624c9804db1a4146 Mon Sep 17 00:00:00 2001 From: MitchBred Date: Thu, 6 Aug 2026 14:11:10 +0200 Subject: [PATCH 2/2] ci: lint via het composer-script in plaats van de phplint action De CI op main staat rood sinds dependabot overtrue/phplint van 9.5 naar 9.8 bumpte in #60. De action print zelf "[OK] 24 files" en eindigt daarna alsnog met een non-zero exit, zonder annotatie. De versie die draait meldt zich als 9.8.x-dev, want @9.8 is een meebewegende branch en geen vaste release. Dat betekent dat de lint op elk moment kan omvallen zonder dat er iets in deze repo verandert. Draai phplint daarom via het lint-script, op de versie die het pakket zelf in require-dev pint en op de PHP-versie die de rest van de CI ook gebruikt. Scheelt een third-party action in de keten en geeft dezelfde 24 bestanden. De job heet nu lint in plaats van build, wat beter beschrijft wat hij doet en niet botst met de matrix-jobs in de andere workflow. Branch protection op main heeft geen verplichte checks, dus de hernoeming blokkeert niets. De trigger stond op kale push, wat betekende dat hij ook draaide op branches zonder PR. Nu main plus pull requests, gelijk aan de andere workflow. Co-Authored-By: Claude Opus 5 (1M context) --- .github/workflows/lint.yml | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 4697bea8..5dd58930 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -1,14 +1,25 @@ name: CI -on: push +on: + push: + branches: [ main ] + pull_request: + jobs: - build: + lint: runs-on: ubuntu-latest steps: - name: Checkout code uses: actions/checkout@v7 - - name: Checking PHP syntax error - uses: overtrue/phplint@9.8 + + - name: Setup PHP + uses: shivammathur/setup-php@v2 with: - path: . - options: --exclude=*.log + php-version: '8.4' + coverage: none + + - name: Install dependencies + run: composer install --prefer-dist --no-progress --no-interaction + + - name: Checking PHP syntax error + run: composer run-script lint