Skip to content

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

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

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

Conversation

@ash2k

@ash2k ash2k commented Jun 7, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

This is a follow up for #3198 to fix a few more modules. Smaller PR this time, I haven't addressed all modules to reduce the size.

Why is it important?

Related issues

@ash2k ash2k requested a review from a team as a code owner June 7, 2025 05:00
@netlify

netlify Bot commented Jun 7, 2025

Copy link
Copy Markdown

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 77de0cd
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/6843c759af980000083b8502
😎 Deploy Preview https://deploy-preview-3203--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 merged commit 27b2436 into testcontainers:main Jun 7, 2025
22 checks passed
@ash2k ash2k deleted the use-mapped-port branch June 7, 2025 11:54
@mdelapenya mdelapenya added the bug An issue with the library label Jun 8, 2025
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Jun 10, 2025
* main:
  fix: use PortEndpoint() in a few more modules (testcontainers#3203)
  fix: try to fix more IPv6 handling issues (testcontainers#3198)
  chore!: do not wait for all the exposed ports to be ready (testcontainers#3199)
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