Skip to content

Node.js環境構築キット(macOS)更新#16

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

Node.js環境構築キット(macOS)更新#16
Auron0330 wants to merge 11 commits into
masterfrom
add-asdf-node-mac

Conversation

@Auron0330

Copy link
Copy Markdown
Contributor

asdfを使用するように更新

@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.

3点ほどコメントさせていただきました、確認お願いします

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

今現在の最新バージョンを明記しておいた方がいいのかなと思いました。
以下のように

以下のようにバージョンが表示されれば成功です。(最新バージョン v0.8.1 2021/10/16現在)

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.

2720138 にて修正いたしました。
Node.jsは更新頻度高いので、公式サイトで学習者自身が確認するようにしました。

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.

こちらも 2720138 にて修正いたしました。
メジャーアップデートは必須にし、マイナーアップデートは推奨にしました。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

別の教材などでshellの種類を限定していたりするでしょうか?

限定していないのであれば「bashで実行してください」などshellを指定した方が良いと思いました

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.

すみません、shellはbashであることを前提にしていますが、確認方法や切り替え方法は別の章で執筆予定です。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

@Auron0330 Auron0330 Oct 19, 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.

ありがとうございます! 1681120 にて修正いたしました。

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.

0b6f05c にて修正いたしました。

@Auron0330

Copy link
Copy Markdown
Contributor Author

@44x1carbon
ご確認ありがとうございます!
コメントいただいた箇所、修正いたしましたので、ご確認お願いいたします!

#16 (comment)
リリース前にshellの確認・切替方法もあった方がいいので、優先して作成いたします。

@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.

対応いただいた所確認いたしました。問題ありません!
対応ありがとうございます。

LTSのバージョンがv14→v16に更新されたため、バージョン数を修正しました。
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