Skip to content

fix: use PortEndpoint() in a few more modules#3206

Merged
mdelapenya merged 1 commit into
testcontainers:mainfrom
ash2k:use-mapped-port2
Jun 14, 2025
Merged

fix: use PortEndpoint() in a few more modules#3206
mdelapenya merged 1 commit into
testcontainers:mainfrom
ash2k:use-mapped-port2

Conversation

@ash2k

@ash2k ash2k commented Jun 14, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

Follow up for #3203 with a few more modules.

Why is it important?

Related issues

@ash2k ash2k requested a review from a team as a code owner June 14, 2025 03:52
@netlify

netlify Bot commented Jun 14, 2025

Copy link
Copy Markdown

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 581fc6e
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/684cf2080e4f8500089f6057
😎 Deploy Preview https://deploy-preview-3206--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 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks! 🙇

@mdelapenya mdelapenya added the bug An issue with the library label Jun 14, 2025
@mdelapenya mdelapenya self-assigned this Jun 14, 2025
@mdelapenya mdelapenya merged commit 37d18dc into testcontainers:main Jun 14, 2025
37 checks passed
@ash2k ash2k deleted the use-mapped-port2 branch June 14, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue with the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants