Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 23, 2025

This change optimizes the CI workflow by restricting PHPStan static analysis to run only on PHP 8.4 with the highest dependency versions, reducing unnecessary duplicate analysis while maintaining comprehensive test coverage.

Changes Made

  • Added conditional execution to the PHPStan step in the CI workflow using if: matrix.php == '8.4' && matrix.dependencies == 'highest'
  • PHPStan now runs only once per CI build instead of on all 4 matrix combinations
  • Pest tests continue to run on all PHP versions (8.3, 8.4) and dependency combinations (lowest, highest) to ensure compatibility

Impact

Before: PHPStan ran 4 times per CI build:

  • PHP 8.3 + lowest dependencies
  • PHP 8.3 + highest dependencies
  • PHP 8.4 + lowest dependencies
  • PHP 8.4 + highest dependencies

After: PHPStan runs 1 time per CI build:

  • PHP 8.4 + highest dependencies

This reduces CI execution time while still providing thorough static analysis on the most current PHP version with the latest dependency versions. Test coverage remains unchanged across all supported PHP versions and dependency combinations.

Fixes #8.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: erikgaal <1234268+erikgaal@users.noreply.github.com>
Copilot AI changed the title [WIP] Run PHPStan analysis only on highest dependencies and PHP 8.4 Restrict PHPStan analysis to highest dependencies and PHP 8.4 only Sep 23, 2025
Copilot AI requested a review from erikgaal September 23, 2025 13:20
@erikgaal erikgaal closed this Sep 23, 2025
@erikgaal erikgaal deleted the copilot/fix-8 branch September 23, 2025 13:37
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.

Run PHPStan analysis only on highest dependencies and PHP 8.4

2 participants