fix: use PortEndpoint() in a few more modules by ash2k · Pull Request #3206 · testcontainers/testcontainers-go

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Comments

Conversation

@ash2k

Copy link

Contributor

@ash2k ash2k commented

Jun 14, 2025

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

Copy link

netlify bot commented

Jun 14, 2025

edited

Loading

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.

Copy link

Member

@mdelapenya mdelapenya left a comment

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

Reviewers

@mdelapenya mdelapenya mdelapenya approved these changes

Assignees

@mdelapenya mdelapenya

Labels

bug

An issue with the library

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ash2k @mdelapenya