SEC-2402: Updating version test matrix.

[nchiasson-dgi]

Summary by CodeRabbit

• Chores
  • Updated CI testing matrix: removed outdated Drupal combinations (including 10.4 and 11.1), added Drupal 10.5, 10.6 and 11.2 across multiple PHP versions, and introduced several experimental/leniency-configured test entries.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated the GitHub Actions phpunit workflow matrix: removed Drupal 10.4 and 11.1 entries, replaced a 10.3 include with 10.5, added/expanded Drupal 10.6 and 11.2 combinations across PHP 8.3–8.5, and marked several entries as experimental with adjusted do-leniency flags.

Changes

Cohort / File(s)	Summary
GitHub Actions workflow .github/workflows/phpunit.yml	Edited matrix includes: removed drupal-version: '10.4' and 11.1; replaced a 10.3 include with 10.5; added/expanded 10.6 and 11.2 combinations for PHP 8.3–8.5; annotated several entries as experimental and adjusted do-leniency flags.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• SEC-1684: Fix/update matrix #26 — Edits the same .github/workflows/phpunit.yml test matrix (Drupal/PHP version entries and experimental flags).

Suggested labels

patch

Suggested reviewers

• jordandukart

Poem

🐰 I hopped through YAML rows tonight,
Swapping versions left and right.
Experimental flags I tuck,
So tests may try their luck. 🌱

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: updating the version test matrix in the GitHub Actions workflow file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)

• JIRA integration encountered authorization issues. Please disconnect and reconnect the integration in the CodeRabbit UI.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.github/workflows/phpunit.yml:
- Around line 49-53: The matrix entry that pairs drupal-version: '10.6' with
php-version: '8.5' (the block containing drupal-version, php-version,
experimental, do-leniency) must be removed or replaced because the Docker image
drupal:10.6-php8.5 does not exist; either delete that matrix row entirely or
change it to a valid combination (e.g., set php-version to '8.4' for
drupal-version '10.6' or change drupal-version to '11.2' if you need php-version
'8.5') and keep the experimental/do-leniency flags as needed.

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

.github/workflows/phpunit.yml (1)

34-48: Remove the drupal:10.6-php8.2 configuration entry from the workflow.

The drupal:10.6-php8.2 Docker image does not exist on Docker Hub. Only 10.6-php8.3 and 10.6-php8.4 are available. This configuration will cause workflow failures. Lines 34-39 should be removed to keep only the working 8.3 and 8.4 variants.

[github-actions]

Tag generated by PR: v1.5.0

[adam-vessey]

This appears to be exploding?: https://github.com/discoverygarden/dgi_migrate/actions/runs/21225355647/job/61070755252?pr=178

[...]
Starting job container
  /usr/bin/docker pull drupal:10.5-php8.2
  Error response from daemon: manifest for drupal:10.5-php8.2 not found: manifest unknown: manifest unknown
[...]

[adam-vessey]

Went ahead and updated things: #30