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

Flutterバージョンを3.22.1変更する #208

Merged
merged 12 commits into from
May 29, 2024

Conversation

Yamasaki-pan961
Copy link
Contributor

@Yamasaki-pan961 Yamasaki-pan961 commented May 17, 2024

概要

対応箇所

  • 依存関係の更新
  • ファイル名変更 colors_schemes.g.dart -> colors_schemes.dart
  • 非推奨のColorsScheme.surfaceVariantのマイグレート
  • cocoapodsのバージョンを最新化
  • Android/iOS の作成時のテンプレートに合わせて関連ファイルを更新

レビュー観点

バージョン変更時に変更するべき点が不足していないか

レビューレベル

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

レビュー優先度

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

画像 / 動画

image

動作確認手順

アプリを起動して、正常に動作するか確かめてください

Copy link

github-actions bot commented May 17, 2024

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

https://flutter-mobile-project-template-catalog--pr208-feature-4fds976c.web.app

(expires Wed, 05 Jun 2024 11:47:18 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23

@Yamasaki-pan961 Yamasaki-pan961 changed the title Feature/flutter update 3 22 Flutterバージョンを3.22.0ni May 17, 2024
@Yamasaki-pan961 Yamasaki-pan961 changed the title Flutterバージョンを3.22.0ni Flutterバージョンを3.22.0変更する May 17, 2024
@Yamasaki-pan961 Yamasaki-pan961 marked this pull request as ready for review May 28, 2024 01:07
@yumemi-team-review-requester yumemi-team-review-requester bot requested review from a team, warahiko and Aosanori and removed request for a team May 28, 2024 01:07
Copy link

Ready for review 🚀

melos.yaml Outdated Show resolved Hide resolved
@Aosanori
Copy link
Contributor

@Yamasaki-pan961
動作に関しては問題ないと思います!

Copy link
Contributor

@Aosanori Aosanori left a comment

Choose a reason for hiding this comment

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

ざっと確認いたしましたが、コードレビュー初心者であることと、このPJでは新参者であるため、他の方のレビューもいただけるといいかなという状態です 🙇

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.

@Yamasaki-pan961
いくつかコメントしましたので、ご対応お願いします 🙏
また、 Gradle の更新もしておきたいです。

案件情報の兼ね合いで詳しくは Slack でやり取りさせてください、、!

apps/app/pubspec.yaml Outdated Show resolved Hide resolved
apps/catalog/pubspec.yaml Outdated Show resolved Hide resolved
melos.yaml Outdated Show resolved Hide resolved
packages/cores/core/pubspec.yaml Outdated Show resolved Hide resolved
packages/cores/data/pubspec.yaml Outdated Show resolved Hide resolved
packages/features/debug_mode/pubspec.yaml Outdated Show resolved Hide resolved
packages/features/github_repository/pubspec.yaml Outdated Show resolved Hide resolved
packages/features/setting/pubspec.yaml Outdated Show resolved Hide resolved
packages/features/webview/pubspec.yaml Outdated Show resolved Hide resolved
.fvmrc Outdated Show resolved Hide resolved
@Yamasaki-pan961 Yamasaki-pan961 changed the title Flutterバージョンを3.22.0変更する Flutterバージョンを3.22.1変更する May 28, 2024
@blendthink
Copy link
Member

blendthink commented May 28, 2024

こちら @ Yamasaki-pan961 さんとお話して、 @blendthink が引き継ぐことになりました、、!
明日、対応進めていきます 💪

@blendthink blendthink requested review from morikann and removed request for warahiko May 28, 2024 14:18
@blendthink blendthink requested a review from trm11tkr May 28, 2024 14:18
morikann
morikann previously approved these changes May 29, 2024
Copy link
Contributor

@morikann morikann left a comment

Choose a reason for hiding this comment

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

LGTM です!

📝 mason で作成するパッケージの dart, flutter バージョンは hooks で自動更新されるため、わざわざ更新していない認識です。

@blendthink
Copy link
Member

レビューありがとうございます!

📝 mason で作成するパッケージの dart, flutter バージョンは hooks で自動更新されるため、わざわざ更新していない認識です。

あー、ただ、少し気持ち悪いのでさくっと修正しちゃいましょうかね  🙏

@blendthink
Copy link
Member

↓で対応して、ローカルで動作確認できたので、CI 通ったらマージさせていただきます!
7ade414

@blendthink blendthink requested a review from morikann May 29, 2024 12:08
@blendthink blendthink dismissed morikann’s stale review May 29, 2024 12:08

変更したため

@Yamasaki-pan961 Yamasaki-pan961 merged commit 60afee8 into main May 29, 2024
6 checks passed
@Yamasaki-pan961 Yamasaki-pan961 deleted the feature/flutter-update_3-22 branch May 29, 2024 12:12
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.

Flutter SDK バージョンを 3.22.1 へ更新
4 participants