-
Notifications
You must be signed in to change notification settings - Fork 0
Firebase CLI のインストール手順の修正 #418
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
base: main
Are you sure you want to change the base?
Conversation
Generated by 🚫 Danger |
docs/onboarding/setup/09_Firebase.md
Outdated
| Windowsのデフォルトの設定で「スクリプトの実行」が禁止されているため、以下のコマンドで設定を変更します。 | ||
|
|
||
| ```pwsh | ||
| Set-ExecutionPolicy RemoteSigned -Scope CurrentUser -Force | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この部分のみが Windows のみで必要なステップで、他は macOS & Windows で共通な気がするんだけど、
別々に書かずに、1つにまとめられないかな?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1つにまとめて書いてみます!!
凄い細かい話だとは思うのですが、共通のコマンドを記載するときは、右上に「pwsh」か「zsh」どちらを書けばいいでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
すごくいい視点です
zsh で大丈夫!
macOSを基本にして書いて、Windows独自のものがあればそれを書き足す、というイメージで!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
褒められました!了解です!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macOSとWindowsで共通している工程をまとめて記述してみました!
ご確認よろしくお願いします。
hikaru-0602
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
私に見落としがなければLGTM
masaya-osuga
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
kantacky
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
お待たせしました!
対応ありがとうございます 🙇
概要
09_Firebase.mdにおいて、Windows環境でのセットアップ手順の修正と、Firebaseログイン時の挙動に関する補足説明を追加しました。
やったこと
-
Set-ExecutionPolicyの実行順序をnpm installの前に移動し、Firebase CLI インストールでエラーが発生しない手順に修正しました。firebase login実行時に表示される設定質問(Gemini機能や統計収集)に対し、n(No) を選択する手順を明記しました。関連する Issue
確認したこと
firebase-toolsが正常にインストールできることを確認UI 差分
※ドキュメント修正のため、UI変更はありません。
コメント
firebase login実行時に表示される設定質問は以下のとおりです。information?
nを選択する」と書いたのですが、大丈夫ですか?メモ