search

hazelcast / hz-docs

Source content for the Hazelcast Platform documentation
10 stars 92 forks source link

Remove redundant `docker pull` from installation instructions #1071

Closed JackPGreen closed 3 months ago

JackPGreen commented 3 months ago

The Docker introduction tells you to:

  1. Install Docker
  2. Pull the image
  3. Run the image

But (2) is redundant as docker run hazelcast/hazelcast:5.3.7 will download if required anyway:

Unable to find image 'hazelcast/hazelcast:5.3.7' locally
5.3.7: Pulling from hazelcast/hazelcast
8a49fdb3b6a5: Pull complete
acb6c723f025: Pull complete
dfe629926fe6: Pull complete
610b8e3f6d86: Pull complete
4f4fb700ef54: Pull complete
933e63ec690a: Pull complete
Digest: sha256:b3d2d47e6387f15b28b92a393a2366514ef7160bca00d8617b4306a1b83027c1
Status: Downloaded newer image for hazelcast/hazelcast:5.3.7
netlify[bot] commented 3 months ago

Deploy Preview for hardcore-allen-f5257d ready!

Name Link
Latest commit 957c0d05343dea5784af52853bdac87e210778c8
Latest deploy log https://app.netlify.com/sites/hardcore-allen-f5257d/deploys/6614038a6f78f400080f4eed
Deploy Preview https://deploy-preview-1071--hardcore-allen-f5257d.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Serdaro commented 3 months ago

@JackPGreen , thanks a lot. I am assuming it also applies to https://github.com/JackPGreen/hz-docs/blob/remove-redundant-docker-pull/docs/modules/getting-started/pages/install-enterprise.adoc?

JackPGreen commented 3 months ago

@JackPGreen , thanks a lot. I am assuming it also applies to https://github.com/JackPGreen/hz-docs/blob/remove-redundant-docker-pull/docs/modules/getting-started/pages/install-enterprise.adoc?

Good point

c4e81ccc998c466844b4e2338f0e5a42b576ec14