Skip to content

Conversation

@mdelapenya
Copy link
Member

What does this PR do?

Use the Run function in vearch

Why is it important?

Migrate modules to the new API, improving consistency and leveraging the latest testcontainers functionality.

Related issues

This commit migrates the vearch module to use the new testcontainers.Run() API.

The main changes are:
- Use testcontainers.Run() instead of testcontainers.GenericContainer()
- Convert to moduleOpts pattern with functional options
- Use WithExposedPorts, WithCmd, WithHostConfigModifier, WithFiles, WithWaitStrategy

Ref: testcontainers#3174

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@mdelapenya mdelapenya requested a review from a team as a code owner October 9, 2025 15:07
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Oct 9, 2025
@netlify
Copy link

netlify bot commented Oct 9, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 7c51ea6
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/68e7d47dfff16b000811375f
😎 Deploy Preview https://deploy-preview-3441--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mdelapenya mdelapenya self-assigned this Oct 9, 2025
@mdelapenya mdelapenya merged commit 859df3f into testcontainers:main Oct 9, 2025
15 checks passed
@mdelapenya mdelapenya deleted the use-run-claude-vearch branch October 9, 2025 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant