Skip to content

Fix build-test CI failure by pinning doctrine/instantiator to PHP 8.2-compatible lock version - #1275

Merged
boonebgorges merged 2 commits into
masterfrom
copilot/fix-failing-build-test-job
Aug 6, 2026
Merged

Fix build-test CI failure by pinning doctrine/instantiator to PHP 8.2-compatible lock version#1275
boonebgorges merged 2 commits into
masterfrom
copilot/fix-failing-build-test-job

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

build-test failed during composer install because composer.lock pinned doctrine/instantiator to 2.1.0, which now requires PHP ^8.4, while the workflow runs on PHP 8.2.x. This PR restores lockfile compatibility with the CI runtime.

  • Root cause

    • doctrine/instantiator in composer.lock resolved to 2.1.0 (php: ^8.4), creating an unsatisfiable platform constraint in CI.
  • Targeted lockfile correction

    • Updated only the doctrine/instantiator package entry in composer.lock to 2.0.0 (php: ^8.1).
    • Aligned associated lock metadata (source, dist, support URL, package require-dev, timestamp) to the selected version.
  • Scope

    • No production PHP code changes.
    • No dependency graph expansion; this is a surgical lockfile compatibility correction.
{
  "name": "doctrine/instantiator",
  "version": "2.0.0",
  "require": {
    "php": "^8.1"
  }
}

Co-authored-by: boonebgorges <246627+boonebgorges@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build-test Fix build-test CI failure by pinning doctrine/instantiator to PHP 8.2-compatible lock version Aug 6, 2026
Copilot AI requested a review from boonebgorges August 6, 2026 14:36
@boonebgorges

Copy link
Copy Markdown
Contributor

@copilot Thanks for this. Would it be simpler or more secure if we instead bumped the CI requirement to PHP 8.4+?

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Thanks for this. Would it be simpler or more secure if we instead bumped the CI requirement to PHP 8.4+?

I’d avoid that in this fix. Bumping CI to 8.4 would stop testing compatibility on the current workflow runtime and could mask breakage for users running older supported PHP versions. The lockfile correction is narrower and keeps the existing compatibility target intact. If we decide to officially raise the project’s minimum PHP version, we can do that in a separate PR and then let the lockfile float forward.

@boonebgorges
boonebgorges marked this pull request as ready for review August 6, 2026 14:39
@boonebgorges
boonebgorges merged commit 94c4f45 into master Aug 6, 2026
1 check 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