Remove redundant lifetime parameters

[eharcevs]

Is the 'd lifetime parameter in Container and ContainerAsync redundant? The comment says:

Tracks the lifetime of the client to make sure the container is dropped before the client.

This seems unnecessary since the container already holds a pointer (Box) to the client anyway, which ensures that the container will get dropped before the client. Dropping the redundant lifetime parameter results in a simpler API. wdyt?

[neverchanje]

Ran into a similar issue when I was trying to create a global container for all my tests. Maybe #354 would be helpful for you. Since a related fix was merged #384, we can now use the latest (though unreleased) version. That could probably solve your problem.

testcontainers = { git = "https://github.com/testcontainers/testcontainers-rs.git", rev = "466f503" }

#[macro_use]
extern crate lazy_static;

lazy_static! {
    static ref DOCKER: Cli = Cli::default();
    static ref NODE: Container<'static, MySQL> = DOCKER.run(MySQL::new());
}

[thomaseizinger]

Thank you for the PR!

I'd like to move the API towards #386 instead which will also remove this lifetime in favor of runtime reference counting.