Skip to content

PHP8.0環境構築キット(Windows10)追加#21

Open
Auron0330 wants to merge 7 commits into
masterfrom
add-asdf-php-windows
Open

PHP8.0環境構築キット(Windows10)追加#21
Auron0330 wants to merge 7 commits into
masterfrom
add-asdf-php-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の環境構築手順のリンクを記載してもいいかと思います

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.

動作結果のメッセージが抜けているようです

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) と同様の内容で修正いたしました。
1e94a4f にて対応いたしました。

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.

13864f2 にて対応いたしました。

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

こちら、同時に20個くらいパッケージをインストールしているので、実行結果の行数がかなり多く、インストールの確認が大変なのでここでは省略しています。
エラー発生の頻度はあまり高くないと思うので、問題ないかと思います。

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.

ありがとうございます。
70fca87 にて追記いたしました。

@Auron0330

Copy link
Copy Markdown
Contributor Author

@44x1carbon
ご確認ありがとうございました!
こちら対応いたしましたので、ご確認お願いいたします!

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