Skip to content

Commit aebe9cd

Browse files
committed
Bump to PHP 8.3 minimum, migrate to Mago QA toolchain, add Pdo\Result coverage
php-db/phpdb 0.6.x added getQueryResult() to Driver\ResultInterface, which Sqlite already satisfies for free via the shared core Pdo\Result class (no sqlite-owned Result class exists). Ports the core's exhaustive Pdo\Result unit test suite to cover it, matching the coverage mysql/pgsql carry for their own Result classes. Also replaces laminas-coding-standard/phpstan with the shared php-db/phpdb-qa-tools Mago toolchain, mirroring phpdb-qa-analysis-sql: mago.toml extending the shared base, the qa-tools phpunit.xml.dist template, mago-based composer scripts, and the reusable CI workflow. Pre-existing lint/analyzer findings in untouched code are captured in lint-baseline.toml/analyzer-baseline.toml rather than fixed here.
1 parent 3eeb298 commit aebe9cd

13 files changed

Lines changed: 2332 additions & 603 deletions
Lines changed: 8 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,15 @@
11
name: "Continuous Integration"
22

33
on:
4-
pull_request:
54
push:
6-
branches:
7-
tags:
5+
pull_request:
86

97
jobs:
10-
matrix:
11-
name: Generate job matrix
12-
runs-on: ubuntu-latest
13-
outputs:
14-
matrix: ${{ steps.matrix.outputs.matrix }}
15-
steps:
16-
- name: Gather CI configuration
17-
id: matrix
18-
uses: laminas/laminas-ci-matrix-action@v1
19-
208
qa:
21-
name: QA Checks
22-
needs: [matrix]
23-
runs-on: ${{ matrix.operatingSystem }}
24-
strategy:
25-
fail-fast: false
26-
matrix: ${{ fromJSON(needs.matrix.outputs.matrix) }}
27-
steps:
28-
- name: ${{ matrix.name }}
29-
uses: laminas/laminas-continuous-integration-action@v1
30-
with:
31-
job: ${{ matrix.job }}
9+
uses: php-db/phpdb-qa-tools/.github/workflows/continuous-integration.yml@0.1.x
10+
secrets: inherit
11+
with:
12+
php-versions: '["8.3", "8.4", "8.5"]'
13+
run-integration: true
14+
enable-codecov: false
15+
enable-infection: false

‎.laminas-ci.json‎

Lines changed: 0 additions & 12 deletions
This file was deleted.

0 commit comments

Comments
 (0)