docs: update readme to indicate the port binding

docker-library / docs

Documentation for Docker Official Images in docker-library

https://github.com/docker-library/official-images

MIT License

5.08k stars 2.2k forks source link

docs: update readme to indicate the port binding #2486

Closed felipementel closed 2 months ago

felipementel commented 2 months ago

In order to make it easier for developers who want to run SonarQube in a container, I think it is worth indicating the port binding when running the docker run command.

Our exemple are using the tag 10.6.0-community

andreracz commented 2 months ago

Super useful information.

tianon commented 2 months ago

A few notes:

this should be in content.md, not README.md (the latter is autogenerated, per the comment at the top of it)
it should use %%IMAGE%% for image references
10.6.0 is a really specific image reference that should probably be somewhat relaxed (either 10.6 or 10)
cc sonarqube image maintainers: @carminevassallo @jCOTINEAU @davividal

felipementel commented 2 months ago

@tianon, @whalelines and @alvdavi Can you verify my updates?

jCOTINEAU commented 2 months ago

all good on our side; thanks a lot for taking the time to improve our documentation.

felipementel commented 2 months ago

@tianon @jCOTINEAU @alvdavi

thank you so much!