Skip to content

Node.js環境構築キット(Windows10)追加#17

Open
Auron0330 wants to merge 11 commits into
masterfrom
add-asdf-node-windows
Open

Node.js環境構築キット(Windows10)追加#17
Auron0330 wants to merge 11 commits into
masterfrom
add-asdf-node-windows

Conversation

@Auron0330

Copy link
Copy Markdown
Contributor

No description provided.

@yonyon-web yonyon-web left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

数点コメントしました確認お願いします

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

「まだの方はこちらからWSL2とUbuntuの導入を進めてください」という感じでWSLの導入手順のリンクを貼ってもいいかなと思いました

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.

ありがとうございます。

WSL2の構築キットリリース後に追加します。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

この章では複数の作業が一連の流れで書かれているので1作業毎に h3 で見出しをつけた方が読みやすいかと思いました。

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.

3a5f364 にて対応いたしました。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

自分はコマンドの後に実行結果を書くというフォーマットを知っているので理解できるのですが読者はフォーマットをしらないので
「コマンドが完了しても特にメッセージは表示されません。」の方が意図が伝わりやすいかなと思いました。

Copy link
Copy Markdown

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.

9704dd4 にて追記いたしました。

Copy link
Copy Markdown

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.

#17 (comment) と同様です。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

上でapt installが一度出てきていますが、ここにも実行後の結果を記載した方がいいと思います

@Auron0330 Auron0330 Oct 29, 2021

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.

apt installコマンドはインストールできたかどうかがわかる、はっきりしたメッセージはなく、表示行数も多いので、 acc58a5 にて、数十行表示される旨を追記しました。
インストール後にgpg --versionなどでインストールされたかどうか確認するので、問題ないかと思います。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

PHPの教材では ::: warn で囲われていますね、::: warn の方がわかりやすいと思うので統一してもいいかなと思います。

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.

7f939bb にて対応いたしました。

Copy link
Copy Markdown

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.

こちらすぐ下で記載しています。
ご確認お願いします。

@Auron0330

Copy link
Copy Markdown
Contributor Author

@44x1carbon
コメントいただいた件、対応いたしました。
ご確認お願いいたします!

@yonyon-web yonyon-web left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

修正内容確認しました、問題ないと思います!

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.

2 participants