Skip to content

Conversation

@wimdeblauwe
Copy link
Contributor

This commit adds support for a withFileCopyInclusions method on ComposeContainer and DockerComposeContainer. It allows to specify what files or directories should be copied, instead of just copying all files. If not used, the current behaviour is preserved.

@wimdeblauwe wimdeblauwe requested a review from a team July 4, 2024 13:55
@eddumelendez
Copy link
Member

Hi @wimdeblauwe, thanks for your contribution! I think this is a nice improvement. Do you mind adding some docs here and tests as well, please?

…containers#8847)

This commit adds support for a `withFileCopyInclusions` method on `ComposeContainer` and `DockerComposeContainer`. It allows to specify what files or directories should be copied, instead of just copying all files. If not used, the current behaviour is preserved.
@wimdeblauwe
Copy link
Contributor Author

@eddumelendez Thanks for considering my PR! I have added documentation and tests now.

@eddumelendez
Copy link
Member

eddumelendez commented Jul 13, 2024

great! thanks! Running ./gradlew spotlessApply should fix the build.

@wimdeblauwe
Copy link
Contributor Author

@eddumelendez build is ok now.

@eddumelendez eddumelendez added this to the next milestone Jul 17, 2024
@eddumelendez eddumelendez changed the title Allow configuration of files that are copied for docker compose (#8847) Allow copy specific files to docker compose Jul 17, 2024
@eddumelendez eddumelendez merged commit a321cfa into testcontainers:main Jul 17, 2024
@eddumelendez
Copy link
Member

Thanks for your contribution, @wimdeblauwe ! I made some changes and rename the method.

@wimdeblauwe wimdeblauwe deleted the feature/gh-8847 branch July 17, 2024 08:19
fokion pushed a commit to fokion/testcontainers-java that referenced this pull request Jan 18, 2025
This commit adds support for a `withCopyFilesInContainer` method on `ComposeContainer` and `DockerComposeContainer`. It allows to specify what files or directories should be copied, instead of just copying all files. If not used, the current behavior is preserved.

Fixes testcontainers#8847

---------

Co-authored-by: Eddú Meléndez <eddu.melendez@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants