Skip to content

Add PHP 8.5 and 8.6 to CI matrices - #5

Merged
BenMorel merged 7 commits into
mainfrom
copilot/add-php-8-6-to-matrix
Oct 9, 2026
Merged

BenMorel merged 7 commits into
mainfrom
copilot/add-php-8-6-to-matrix

Conversation

Copilot AI commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Extend the existing PHP test matrices through 8.6, including the missing 8.5 release. Keep PHPStan and coverage selection on PHP 8.5, the latest stable version.

  • Test coverage: Add 8.5 and 8.6 to all database test matrices; run the additional PDO mode combinations on 8.5.
  • Latest-stable checks: Set HIGHEST_PHP_VERSION to 8.5.

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>
Copilot AI changed the title [WIP] Add PHP 8.6 to the php-version matrix Add PHP 8.5 and 8.6 to CI matrices Oct 8, 2026
Copilot AI requested a review from BenMorel October 8, 2026 21:55
@BenMorel

BenMorel commented Oct 8, 2026

Copy link
Copy Markdown
Member

@copilot Also update brick/coding-standard to v6

@codecov

codecov Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.08%. Comparing base (6f3b695) to head (42ac148).

Additional details and impacted files
@@            Coverage Diff            @@
##               main       #5   +/-   ##
=========================================
  Coverage     80.08%   80.08%           
  Complexity      102      102           
=========================================
  Files            17       17           
  Lines           246      246           
=========================================
  Hits            197      197           
  Misses           49       49           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>

Copilot AI commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Also update brick/coding-standard to v6

Updated the reusable workflow to v6 in commit $(git rev-parse --short HEAD).

@BenMorel

BenMorel commented Oct 8, 2026

Copy link
Copy Markdown
Member

@copilot You need to update the ecs compose.json as well

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>

Copilot AI commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

@copilot You need to update the ecs compose.json as well

Updated tools/ecs/composer.json to require brick/coding-standard v6 in commit HEAD.

@BenMorel

BenMorel commented Oct 9, 2026

Copy link
Copy Markdown
Member

@copilot Let's replace HIGHEST_PHP_VERSION with HIGHEST_STABLE_PHP_VERSION

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Let's replace HIGHEST_PHP_VERSION with HIGHEST_STABLE_PHP_VERSION

Renamed the workflow variable and all references to HIGHEST_STABLE_PHP_VERSION in commit HEAD.

@BenMorel

BenMorel commented Oct 9, 2026

Copy link
Copy Markdown
Member

@copilot Fix the ecs failure

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Fix the ecs failure

Removed the extra blank line flagged by ECS v6 in commit HEAD. The local ECS run was blocked because Composer could not authenticate to download dependencies; PHP syntax validation passed.

@BenMorel
BenMorel marked this pull request as ready for review October 9, 2026 19:30
@BenMorel

BenMorel commented Oct 9, 2026

Copy link
Copy Markdown
Member

@copilot Change the ones with:

      - "8.4"
      - "8.5"
      - "8.6"

to

      - "8.5"

Co-authored-by: BenMorel <1952838+BenMorel@users.noreply.github.com>

Copilot AI commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Change the ones with:

      - "8.4"

...

Changed the five database matrices that listed only PHP 8.4/8.5/8.6 to PHP 8.5 only in commit HEAD. The broader MySQL/PDO matrix retains 8.2–8.6.

@BenMorel
BenMorel merged commit 7c3b414 into main Oct 9, 2026
34 checks passed
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.

2 participants