example from readme results in panic with older docker versions

[kaxap]

from README.md:

	nginxC, err := testcontainers.GenericContainer(ctx, testcontainers.GenericContainerRequest{
		ContainerRequest: req,
		Started:          true,
	})
	if err != nil {
		t.Error(err)
	}
	defer nginxC.Terminate(ctx)
	ip, err := nginxC.Host(ctx)

if the call to GenericContainer() fails, then the error reported but test execution is continued until ip, err := nginxC.Host(ctx) where it results in panic due to nil pointer dereference.
I think there should be t.Fatalf("cannot create container: %s, check your docker version", err) instead of t.Error(err)
to reduce possible confusion.

[gianarb]

Hey! Thank you for your report. Can you define old? And can you paste the panic? Thanks

[kaxap]

I have upgraded my Docker right away. It is related to this particular issue with Docker Desktop for Mac, as far as I remember.

[mdelapenya]

There is ongoing work to refactor the docs: #351

I think we could close this one if that one is merged

[mdelapenya]

Let's close this issue, as it seems fixed by the Docker for Mac upgrade. Apart from that, in #353 the examples were updated