Skip to content
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

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

tatsutakein
Copy link
Member

@tatsutakein tatsutakein commented Jul 30, 2024

概要

github repository のサンプルアプリパッケージを追加します。
変更ファイル数が多くなってしまうため一旦はパッケージの追加のみとし、実装は別 PR とする予定です。

レビュー観点

  • melos bootstrap ができることをご確認いただきたいです
  • 後続の作業がありますが、追加だけでも変更ファイル数が多くなったため一旦このパッケージ追加のみの PR とさせてください 🙏

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

Before After Design

確認したこと

  • melos bootstrap が成功すること
  • vscode から samples-github-app が実行できること
  • intellij 系の ide から samples-github-app が実行できること

動作確認手順

  • melos bootstrap を実行
  • vscode, intellij 系の ide から samples-github-app を実行

備考

Copy link

github-actions bot commented Jul 30, 2024

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

@tatsutakein tatsutakein marked this pull request as ready for review July 30, 2024 08:30
@yumemi-team-review-requester yumemi-team-review-requester bot requested review from a team, blendthink and trm11tkr and removed request for a team July 30, 2024 08:31
Copy link

Ready for review 🚀

Base automatically changed from improve/GH-275 to main July 30, 2024 09:02
@tatsutakein tatsutakein force-pushed the improve/GH-275-add-github-app-package branch from 3f9f5e2 to 72e3f23 Compare July 30, 2024 09:03
Comment on lines +38 to +41
- label: "@samples/github_app"
files:
- samples/github_app/.+

Copy link
Member Author

Choose a reason for hiding this comment

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

memo-badge

melos bs 時に生成されています。

Comment on lines +77 to +80
- name: "@samples/github_app"
color: 73d994
description: samples github_app package

Copy link
Member Author

Choose a reason for hiding this comment

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

memo-badge

melos bs 時に生成されています。

@@ -6,6 +6,7 @@ packages:
- packages/cores/*
- packages/features/*
- packages/samples/*
- samples/*
Copy link
Member Author

Choose a reason for hiding this comment

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

memo-badge

新しく sample ディレクトリを認知するよう追加しています。

Copy link
Member

@blendthink blendthink left a 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 では難しい?)

Copy link
Member

@blendthink blendthink left a 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 などの対応はしない)

@tatsutakein
Copy link
Member Author

@blendthink
ハドルでお話した改善点について、対応してみました!
お手数ですが、再度ご確認いただけますと幸いです 🙏

@tatsutakein tatsutakein requested a review from blendthink July 31, 2024 03:36
Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

LGTM です!
ご対応ありがとうございます!

@tatsutakein
Copy link
Member Author

丁寧にご確認いただきありがとうございました!
マージいたします 🙆

@tatsutakein tatsutakein merged commit ebbbab7 into main Jul 31, 2024
14 checks passed
@tatsutakein tatsutakein deleted the improve/GH-275-add-github-app-package branch July 31, 2024 05:25
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