-
Notifications
You must be signed in to change notification settings - Fork 142
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
feat: add support for user-configured labels #756
feat: add support for user-configured labels #756
Conversation
✅ Deploy Preview for testcontainers-rust ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
01154df
to
e97fce6
Compare
e97fce6
to
2a458f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you a lot for the contribution! 🙏
I find this pretty small and simple PR looking good to me, just a few comments to discuss and LGTM! ✅
Absolutely my pleasure 😁
I replied to both comments, and pushed the suggested changes. If the responses (and changes) make sense and to you look good, I'll leave the |
Perfect, LGTM ✅ I'll take a look at the second PR soon, thanks one more time! ❤️ |
PR adds support for applying user-configured labels to containers started by
testcontainers
.Note
This PR is a precursor to future support of reusable containers