Skip to content

ci(phpstan): use a matrix for phpstan jobs#1030

Merged
JohnVillalovos merged 1 commit intodevelopfrom
jlvillal/phpstan_performance
Feb 16, 2026
Merged

ci(phpstan): use a matrix for phpstan jobs#1030
JohnVillalovos merged 1 commit intodevelopfrom
jlvillal/phpstan_performance

Conversation

@JohnVillalovos
Copy link
Collaborator

This is to speed it up more.

Copilot AI review requested due to automatic review settings February 16, 2026 16:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Speeds up the PHP static analysis CI by splitting PHPStan runs into parallel matrix jobs while keeping their caches isolated.

Changes:

  • Convert the analyse-php job to a matrix with separate “base” and “next” PHPStan runs.
  • Add per-variant cache directories/keys to avoid cache collisions between the two analyses.
  • Replace two sequential PHPStan steps with a single matrix-driven composer script invocation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/phpstan_performance branch from 2039eca to 0373e2c Compare February 16, 2026 16:36
This is to speed it up more.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JohnVillalovos JohnVillalovos merged commit c15bd82 into develop Feb 16, 2026
19 checks passed
@JohnVillalovos JohnVillalovos deleted the jlvillal/phpstan_performance branch February 16, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant