-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Allow copy specific files to docker compose #8848
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow copy specific files to docker compose #8848
Conversation
b1d39fd to
45a2203
Compare
|
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.
45a2203 to
9ba8a33
Compare
|
@eddumelendez Thanks for considering my PR! I have added documentation and tests now. |
|
great! thanks! Running |
1e89afd to
fa1cacd
Compare
|
@eddumelendez build is ok now. |
|
Thanks for your contribution, @wimdeblauwe ! I made some changes and rename the method. |
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>
This commit adds support for a
withFileCopyInclusionsmethod onComposeContainerandDockerComposeContainer. 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.