Skip to content

Conversation

@DCjanus
Copy link
Contributor

@DCjanus DCjanus commented Nov 18, 2025

Close #786

@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for testcontainers-rust ready!

Name Link
🔨 Latest commit 5b8c797
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-rust/deploys/6928632297516700082d22f4
😎 Deploy Preview https://deploy-preview-878--testcontainers-rust.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.

@DCjanus DCjanus marked this pull request as draft November 18, 2025 07:25
@DCjanus DCjanus marked this pull request as ready for review November 23, 2025 12:52
@DCjanus DCjanus marked this pull request as draft November 27, 2025 15:22
@DDtKey
Copy link
Contributor

DDtKey commented Nov 28, 2025

Hi @DCjanus
Are we good to mark as ready to review and merge? or you have plans to modify something?
Recursive mode can land in separate PR if you wish

@DCjanus DCjanus marked this pull request as ready for review November 28, 2025 08:14
@DCjanus
Copy link
Contributor Author

DCjanus commented Nov 28, 2025

I initially planned to add recursive mode in this PR.
However, after a brief exploration last night, it became clear that the effort required is greater than I expected.

I’m marking this PR as Ready for Review , and will address recursive mode in a future PR.

Thank you sincerely for your time and effort.

@DDtKey DDtKey merged commit 3c04122 into testcontainers:main Nov 29, 2025
18 checks passed
@DCjanus DCjanus deleted the feat/close-786 branch November 29, 2025 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature request] Add file permission to CopyToContainer

3 participants