-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add support for tc.host in ~/.testcontainers.properties #38
base: main
Are you sure you want to change the base?
Conversation
eddumelendez
commented
Feb 16, 2024
- Support tc.host
Read `tc.host` property from `~/.testcontainers.properties` file.
Is the The |
I think we should support both |
Great! @guilleiguaran LMK once it is available and I can rework the PR.
|
While not currently an env var, I think it would be useful to be able to use it. For example, instead of: tc_host = properties.get(:"tc.host") We could have: tc_host = ENV["TESTCONTAINERS_HOST"] || properties.get(:"tc.host") Or swap the order of the two depending on which value you'd like to take precedence. Let me know what you think! :) |
Thanks for the feedback and sorry for not providing the context when submitting the PR. The support for I think what is related to the goal of the discussion here, is supporting |
Hi @guilleiguaran, any update to support reading properties files? |