Skip to content

feat(php): switch to fernapi/php-seed image for wire tests#11816

Merged
fern-support merged 1 commit intomainfrom
devin/1769482651-php-seed-wire-tests
Jan 27, 2026
Merged

feat(php): switch to fernapi/php-seed image for wire tests#11816
fern-support merged 1 commit intomainfrom
devin/1769482651-php-seed-wire-tests

Conversation

@jsklan
Copy link
Copy Markdown
Contributor

@jsklan jsklan commented Jan 27, 2026

Description

Refs: #11778
Requested by @jsklan

Link to Devin run: https://app.devin.ai/sessions/4204d1a51d914df48ba3664351fb9577

This is the follow-up PR to #11778. Now that the fernapi/php-seed Docker image has been built and pushed to DockerHub, this PR switches the PHP seed tests to use it and removes exhaustive:wire-tests from the allowed failures list.

Changes Made

  • Changed scripts image from composer:2.7.9 to fernapi/php-seed (which includes Docker-in-Docker support for running WireMock containers)
  • Removed exhaustive:wire-tests from allowedFailures since it should now pass with the new image

Testing

  • Verified fernapi/php-seed:latest image exists on DockerHub
  • CI will validate that exhaustive:wire-tests passes with the new image

Human Review Checklist

  • Watch CI to confirm exhaustive:wire-tests passes (if it fails, we may need to investigate the php-seed image)

Co-Authored-By: judah@buildwithfern.com <jsklan.development@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@fern-support fern-support merged commit 8c95855 into main Jan 27, 2026
122 of 123 checks passed
@fern-support fern-support deleted the devin/1769482651-php-seed-wire-tests branch January 27, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants