Skip to content
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

Merged
merged 4 commits into from
Jul 17, 2024

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 as a code owner 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
102 checks passed
@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
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