feat!: add container startup timeout with default of 1 minute

testcontainers / testcontainers-rs

A library for integration-testing against docker containers from within Rust.

https://rust.testcontainers.org

Apache License 2.0

745 stars 142 forks source link

Closed DDtKey closed 5 months ago

DDtKey commented 5 months ago

Closes #247

Note: breaking change because of default timeout

For now, ENV variable to configure global timeout isn't introduced. First of all, that's quote similar to other languages, (e.g Go, Java)

Another question is which one should take precedence: ENV over with_startup_timeout or vice versa?

We can extend it later in compatible way, so not that important at the moment.

netlify[bot] commented 5 months ago

Name	Link
Latest commit	2a736e104ef0ea02f091ac5edc2a899177bd31c7
Latest deploy log	https://app.netlify.com/sites/testcontainers-rust/deploys/66537d25321a79000868c81f
Deploy Preview	https://deploy-preview-643--testcontainers-rust.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.