Artem Medvedev

Results 183 comments of Artem Medvedev

> Will you also accept file name argument when doing stdin? Because otherwise you will not have enough information to display in error messages. And I think without file path...

I believe the correct way to fix that is to introduce fallible `try_append` method. It's too fragile to rely on reservation methods & documentation only and easy to miss the...

Hi @Donemmanuelo 👋 Perhaps I misunderstood something, but some quick answers: - Testcontainers support volumes configuration via [`with_mount`](https://docs.rs/testcontainers/0.23.3/testcontainers/core/trait.ImageExt.html#tymethod.with_mount) - you can override this as you wish - What do you...

Could you point me to particular docker feature you're missing? (so we will be able to track each of them separately if any) - "bind mount" is supported as mentioned...

Closed in favor of updated branch - #864

Hi there 👋 Thanks for the detailed description of the issue. This is actually a good question. Since we allow custom consumers, this can result in significant delays per drop...

> Currently, dropping the container requests deletion right away, and I think there may be a race between log streams ending due to container deletion versus log streams ending due...

@surister hi 👋 Thank you for the contribution first of all! Could you take a look at comments from @CommanderStorm ? I find them reasonable and easy to fix

Hi 👋 Perhaps, you need to use [get_host](https://docs.rs/testcontainers/0.23.1/testcontainers/core/struct.ContainerAsync.html#method.get_host) instead of hardcoded localhost. Which is appropriate when container isn't running locally, but remote Could you try? And if it doesn't help,...