Skip to content

Commit e9a9e6f

Browse files
authored
Refactor PHP workflow and update version matrix
1 parent 21889d4 commit e9a9e6f

1 file changed

Lines changed: 5 additions & 9 deletions

File tree

.github/workflows/php.yml

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ name: CI
33
on:
44
push:
55
branches: [ master ]
6-
# Sin filtro de ramas: un PR hacia una rama feature (como #275 -> feat/...)
7-
# también debe correr los tests antes de mergearse.
86
pull_request:
97
workflow_dispatch:
108

@@ -15,16 +13,16 @@ concurrency:
1513

1614
jobs:
1715
# --------------------------------------------------------------------------
18-
# Tests deterministas (sin red). Esto es lo que bloquea el merge.
16+
# Tests deterministas (sin red).
1917
# --------------------------------------------------------------------------
2018
unit:
2119
name: Unit · PHP ${{ matrix.php }}
22-
runs-on: ubuntu-24.04 # ubuntu-22.04 entra en deprecación el 17-09-2026
20+
runs-on: ubuntu-24.04
2321
timeout-minutes: 15
2422
strategy:
25-
fail-fast: false # que el job 7.4 no se cancele si falla el 8.x
23+
fail-fast: false
2624
matrix:
27-
php: [ '7.4', '8.0', '8.1', '8.2', '8.3', '8.4' ]
25+
php: [ '7.4', '8.2', '8.4' ]
2826
include:
2927
- php: '7.4'
3028
coverage: true
@@ -39,8 +37,6 @@ jobs:
3937
extensions: mbstring, curl, soap, dom, xsl, zip, :fileinfo
4038
# pcov solo donde se reporta cobertura; en el resto 'none' (más rápido)
4139
coverage: ${{ matrix.coverage && 'pcov' || 'none' }}
42-
# psalm como PHAR: vimeo/psalm ~5.10 en require-dev exige PHP <= 8.2
43-
# e impide instalar dependencias (y probar) en 8.3/8.4.
4440
tools: composer:v2, psalm:5
4541

4642
- name: Validate composer.json
@@ -78,7 +74,7 @@ jobs:
7874

7975
# --------------------------------------------------------------------------
8076
# Tests de integración contra SUNAT beta / GRE (nubefact). Dependen de
81-
# servicios externos: se ejecutan aparte y NO bloquean el merge.
77+
# servicios externos: se ejecutan aparte.
8278
# --------------------------------------------------------------------------
8379
integration:
8480
name: Integración SUNAT beta (informativo)

0 commit comments

Comments
 (0)