Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Dockerの記事 #29

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

[WIP] Dockerの記事 #29

wants to merge 18 commits into from

Conversation

coord-e
Copy link
Contributor

@coord-e coord-e commented Mar 17, 2019

Dockerについての記事です。

先に #28 をマージしてこれにもtextlintをかけたほうがよさそう

@anharu2394
Copy link
Member

動作確認しました、オッケーです。
内容ですが
最初の方がすごく気難しくて初心者向けではないなと思いました。
あまり初心者を想定していないのならいいですが。
初心者向けにしたいなら、最初はもっと噛み砕いてせち、説明して、詳しいことは最後に説明すればいいのではないでしょうか。

実践的にやってみるところはとても面白そうでワクワクしますね。いいと思います。

textlintのプルリクがマージされlintされたら、Approveとします。

@coord-e
Copy link
Contributor Author

coord-e commented Mar 23, 2019

@anharu2394
とりあえず図は足します。文体はともかく内容として、何が役に立つのかの説明は先に必要だと思っているのですが、どうでしょうか なお対象読者については初めに記してあります

ただ長くて飽きやすい自覚はあるので、図を足しても様子が変わらなかったら何か考え直す必要があるかもしれません

@Sakuten/moderator もし余裕があれば他の方にも見ていただきたいです 特にユースケースの説明がこれで最適である気がしない

@coord-e
Copy link
Contributor Author

coord-e commented Mar 23, 2019

  • textlint
    • とは? に 飽きないように
    • コンテナの状態について特に
  • 参考文献リスト更新(冒頭と一致するよう)
  • 前の記事のリンクを初めに置く(できれば一覧?)
  • 権限はどうこう、の位置が違うかな
  • run -p の説明(リソース分離と絡めて)
  • SakutenDev内のコードへのリンク

@coord-e coord-e changed the title Dockerの記事 [WIP] Dockerの記事 Mar 23, 2019
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.

Docker
2 participants