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: GitHub 関連の実装を apps/app から samples/github_app に移動 #297

Merged
merged 5 commits into from
Aug 1, 2024

Conversation

tatsutakein
Copy link
Member

@tatsutakein tatsutakein commented Jul 31, 2024

概要

GitHub 関連の実装を apps/app から samples/github_app に移動します。
クリーンアップワークフローまわりは別 PR とさせてください。

  • apps/app
    • ホームタブの画面を github repository 一覧から中央テキスト配置に変更
    • samples_github_repository への依存を削除
  • samples/github_app
    • flutter_lints -> yumemi_lints に変更
    • github repository のサンプルアプリ実装を追加

レビュー観点

  • 実装に過不足がないか
  • クリーンアップワークフローまわりは別 PR とさせてください 🙏

レビューレベル

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

レビュー優先度

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

画像 / 動画

apps/app

CleanShot.2024-07-31.at.15.33.40.mp4

samples/github_app

CleanShot.2024-07-31.at.15.38.09.mp4

確認したこと

  • apps/app がビルドできること
  • apps/app のホーム画面が中央にテキスト配置に変更されていること
  • samples/github_app で GitHub 関連の処理が追加されていること

動作確認手順

  • melos bootstrap を実行
  • apps/app を android or ios で実行できること
  • samples/github_app を web で実行できること

備考

@github-actions github-actions bot added @apps/app Application development @samples/github_app samples github_app package labels Jul 31, 2024
Copy link

github-actions bot commented Jul 31, 2024

Visit the preview URL for this PR (updated for commit 5839268):

https://flutter-mobile-project-template-catalog--pr297-improve-mv4q8r7x.web.app

(expires Thu, 08 Aug 2024 08:22:17 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23

@tatsutakein tatsutakein marked this pull request as ready for review July 31, 2024 06:44
@yumemi-team-review-requester yumemi-team-review-requester bot requested review from a team, Aosanori and mqkotoo and removed request for a team July 31, 2024 06:45
Copy link

Ready for review 🚀

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 です!
ご対応ありがとうございます!

@blendthink blendthink enabled auto-merge August 1, 2024 08:20
@blendthink blendthink merged commit 1fc8a56 into main Aug 1, 2024
13 checks passed
@blendthink blendthink deleted the improve/GH-275-github-app-impl branch August 1, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@apps/app Application development @samples/github_app samples github_app package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants