Skip to content

Conversation

khs1994
Copy link
Collaborator

@khs1994 khs1994 commented Dec 8, 2017

Proposed changes (Mandatory)

Add docker config Only Swarm mode #283

Add LinuxKit #287

Fix issues (Optional)

Remove useless file

error in SUMMARY.md

@khs1994 khs1994 requested a review from yeasy December 8, 2017 15:04
@khs1994 khs1994 self-assigned this Dec 8, 2017
@khs1994
Copy link
Collaborator Author

khs1994 commented Dec 8, 2017

WaitFish 的《Docker学习手册v1.0》已归档到新的分支

@khs1994 khs1994 changed the title Update content Release v0.9-rc2 Dec 8, 2017
@@ -1,49 +0,0 @@
# Some useful commands to use docker.
# Author: yeasy@github
# Created:2014-09-25
Copy link
Owner

Choose a reason for hiding this comment

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

Would like to suggest keep this file, as we wanna add some quick cmd alias if possible.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

该文件不是用于 进入容器一节的吗,进入容器一节已经不需要该文件了

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

没必要弄些额外的脚本,尽量用 docker 原始命令

@@ -1,126 +0,0 @@
# Docker命令查询
Copy link
Owner

Choose a reason for hiding this comment

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

Why need to delete this file? Instead, we should update its content, for quick cmd usage reference.

Copy link
Collaborator Author

@khs1994 khs1994 Dec 9, 2017

Choose a reason for hiding this comment

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

https://github.com/yeasy/docker_practice/tree/master/appendix/command

与这个文件重复了,SUMMARY 包含的是这个,没包含根目录的 appendix_command/README.md

@yeasy yeasy merged commit 0c594a5 into yeasy:master Dec 9, 2017
@yeasy yeasy added this to the v0.9.0 milestone Dec 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants