-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat(samples): github repository のサンプルアプリパッケージを追加 #295
Conversation
Visit the preview URL for this PR (updated for commit 0440da1): https://flutter-mobile-project-template-catalog--pr295-improve-4id0rq6n.web.app (expires Wed, 07 Aug 2024 03:32:46 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23 |
Ready for review 🚀 |
3f9f5e2
to
72e3f23
Compare
- label: "@samples/github_app" | ||
files: | ||
- samples/github_app/.+ | ||
|
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.
- name: "@samples/github_app" | ||
color: 73d994 | ||
description: samples github_app package | ||
|
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.
@@ -6,6 +6,7 @@ packages: | |||
- packages/cores/* | |||
- packages/features/* | |||
- packages/samples/* | |||
- samples/* |
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.
LGTM です!
サンプルアプリの実装をする際に IDE の起動設定ファイルの変更もあわせて対応したいですね、、! 他のサンプルが増えたときにコマンドが多くなりすぎないように、できれば VS Code は input をうまく活用したいかも?( IntelliJ では難しい?)
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.
📝 Slack のハドルでのお話メモ
- Android・iOS アプリは削除して Web アプリだけにする
- IDE の設定ファイルの更新をする( input などの対応はしない)
@blendthink |
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 です!
ご対応ありがとうございます!
丁寧にご確認いただきありがとうございました! |
概要
github repository のサンプルアプリパッケージを追加します。
変更ファイル数が多くなってしまうため一旦はパッケージの追加のみとし、実装は別 PR とする予定です。
レビュー観点
レビューレベル
レビュー優先度
画像 / 動画
確認したこと
動作確認手順
備考