Skip to content

Fixes broken Docker documentation link - #735

Merged
KostyaSha merged 1 commit into
docker-java:masterfrom
mfulgo:patch-1
Jan 26, 2017
Merged

KostyaSha merged 1 commit into
docker-java:masterfrom
mfulgo:patch-1

Conversation

@mfulgo

@mfulgo mfulgo commented Nov 2, 2016

Copy link
Copy Markdown
Contributor

The official Docker documentation has moved around a bit and the link for how to configure the Docker Engine needed to be updated. The previous link pointed to a page that no longer exists.


This change is Reviewable

The official Docker documentation has moved around a bit and the link for how to configure the Docker Engine needed to be updated. The previous link pointed to a page that no longer exists.
Comment thread README.md
DOCKER_OPTS="-H tcp://127.0.0.1:2375 -H unix:///var/run/docker.sock"

More details about setting up Docker server can be found in official documentation: http://docs.docker.io/en/latest/use/basics/
More details about setting up Docker Engine can be found in the official documentation: https://docs.docker.com/engine/admin/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, but this page specifically talks about configuration and things like setting DOCKER_OPTS, which is what I think most people would want.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@codecov-io

Copy link
Copy Markdown

Current coverage is 71.13% (diff: 100%)

Merging #735 into master will decrease coverage by 0.13%

@@             master       #735   diff @@
==========================================
  Files           303        303          
  Lines          6547       6547          
  Methods           0          0          
  Messages          0          0          
  Branches        592        592          
==========================================
- Hits           4666       4657     -9   
- Misses         1601       1607     +6   
- Partials        280        283     +3   

Powered by Codecov. Last update 75e9a2c...39ef5d1

@KostyaSha KostyaSha added this to the 3.0.8 milestone Jan 26, 2017
@KostyaSha
KostyaSha merged commit 5323605 into docker-java:master Jan 26, 2017
@mfulgo
mfulgo deleted the patch-1 branch January 31, 2017 18:45
panuse pushed a commit to TuKangTech/docker-java that referenced this pull request Aug 20, 2017
Fixes broken Docker documentation link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants