Skip to content

PHP8.0環境構築キット(macOS)追加#18

Open
Auron0330 wants to merge 9 commits into
masterfrom
add-asdf-php-mac
Open

PHP8.0環境構築キット(macOS)追加#18
Auron0330 wants to merge 9 commits into
masterfrom
add-asdf-php-mac

Conversation

@Auron0330

Copy link
Copy Markdown
Contributor

環境により、環境変数の追加が必要かもしれないです。

こちらの環境では必要ありませんでした。

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

Homebrewのインストールは他の環境構築でも使っているので別ページにしてリンクにした方がいいと思いました。
それぞれ埋め込んでいると変更が会った時に修正のコストが高くなりますし

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

コメントありがとうございます。

確かに共通部分はまとめた方がメンテナンス性はいいのですが、あちこちにキットを用意すると執筆者が必要なものが探しづらくなったり、漏れが発生したりする懸念があるので、言語ごとにまとめた方が、執筆者にとっては楽かなと思い、この構成にしています。

また、本リポジトリは学習者向けではなく、執筆者にこのキットをコピーしてもらい、以下のように教材の1パートとして使用してもらっています。
https://www.techpit.jp/courses/2/curriculums/3/sections/12/parts/40
なのでキット内に本リポジトリのリンクは貼れないかなと思います。

と言っても何か解決策があればと思うのですが、何か方法があればご提案いただけると幸いです。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

なるほど、内容をコピーして使うのですね承知しました。

修正コストが高くなる問題は解決できないですが、GitHubの検索で修正箇所を洗い出せるので特に問題ないかと思います!

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

asdfのインストールは他の環境構築でも使っているので別ページにしてリンクにした方がいいと思いました。
それぞれ埋め込んでいると変更が会った時に修正のコストが高くなりますし

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.

こちら、同時に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.

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

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.

c62ac80 にて対応いたしました。

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.

031de2a にて対応いたしました。

@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