Skip to content

ci(refactor): move phpunit to its own YAML file#1031

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

ci(refactor): move phpunit to its own YAML file#1031
JohnVillalovos merged 1 commit intodevelopfrom
jlvillal/phpstan_performance

Conversation

@JohnVillalovos
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings February 16, 2026 16:49
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

This PR refactors GitHub Actions CI by extracting the PHPUnit job into its own dedicated workflow file, separating unit tests from linting/analysis tasks.

Changes:

  • Added a standalone .github/workflows/phpunit.yml workflow to run PHPUnit across a PHP version matrix.
  • Removed the embedded phpunit job from .github/workflows/lint-and-analyse-php.yml to keep that workflow focused on linting/static analysis.

Reviewed changes

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

File Description
.github/workflows/phpunit.yml New workflow that runs unit tests (PHPUnit) on PRs targeting develop, with a PHP 8.2/8.3/8.4 matrix.
.github/workflows/lint-and-analyse-php.yml Deletes the previously inlined PHPUnit job now handled by the new dedicated workflow.

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

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/phpstan_performance branch from cee41e2 to 08f5e3e Compare February 16, 2026 17:40
@JohnVillalovos JohnVillalovos merged commit cd5e596 into develop Feb 16, 2026
13 checks passed
@JohnVillalovos JohnVillalovos deleted the jlvillal/phpstan_performance branch February 16, 2026 17:43
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