build: パッケージ依存管理部分を melos -> pub workspaces へ移行 #371

tatsutakein · 2025-01-13T10:47:46Z

概要

close: [Feature]: Pub Workspace を使用する #362

パッケージ依存管理部分を melos -> pub workspaces へ移行します。
また、すぐに安定版になると見込み pub workspaces と親和性が高くなる melos 7.0.0-dev.3 へも併せて移行します。

やったこと

theme_extensions_builder 4.0.0-> 3.1.0 にダウングレード
- 間接的に依存している source_gen のメジャーバージョンが異なっていたため
パッケージ依存管理部分を melos -> pub workspaces へ移行
melos ^6.3.0 -> ^7.0.0-dev.3 にバージョンアップ

レビュー観点

コードに違和感がないか
変更内容に過不足がないか

レビューレベル

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

レビュー優先度

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

画像 / 動画

見た目に関する変更がないため省略します。

確認したこと

Android, iOS 向けの apps/app ビルドが実行できること
Web, MacOS 向けの apps/catalog ビルドが実行できること

動作確認手順

以下のコマンドを実行して melos 6 系までで生成される pubspec_overrides.yaml を削除

find . -name "pubspec_overrides.yaml" -type f -delete

以下のコマンドを実行して melos 7 系の cli を activate

dart pub global activate melos 7.0.0-dev.3

以下のコマンドを実行して melos bootstrap を実行

melos bs

以下のビルドができることの確認

Android, iOS 向けの apps/app ビルド
Web, MacOS 向けの apps/catalog ビルド

備考

以下を参考に実装しています。

github-actions · 2025-01-13T10:50:21Z

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

https://flutter-mobile-project-template-catalog--pr371-feature-8yzuc9lv.web.app

_{(expires Fri, 24 Jan 2025 12:14:52 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23}

tatsutakein · 2025-01-13T11:36:10Z

packages/cores/designsystem/pubspec.yaml

+  theme_extensions_builder_annotation: ^3.1.0

 dev_dependencies:
  build_runner: ^2.4.14
  flutter_gen_runner: ^5.8.0
  flutter_test:
    sdk: flutter
-  theme_extensions_builder: ^4.0.0
+  theme_extensions_builder: ^3.1.0


間接的に依存している source_gen のメジャーバージョンが異なってしまうためダウングレードします。

-> flutter pub get Resolving dependencies... Because cores_designsystem depends on theme_extensions_builder ^4.0.0 which depends on source_gen ^2.0.0, source_gen ^2.0.0 is required. And because widgetbook_generator 3.9.1 depends on source_gen ^1.1.0, widgetbook_generator 3.9.1 is forbidden. Because no versions of widgetbook_generator match >3.9.1 <4.0.0 and catalog depends on widgetbook_generator ^3.9.1, widgetbook_generator 3.9.1 is required. Thus, version solving failed. Failed to update packages.

tatsutakein · 2025-01-13T11:39:10Z

pubspec.yaml

 dev_dependencies:
  mason_cli: ^0.1.0-dev.55
  melos: ^6.3.0
  path: ^1.9.1
  yaml: ^3.1.3
  yaml_edit: ^2.2.2
+
+dependency_overrides:
+  file: ^7.0.0


yumemi_lints と melos で競合していたため。

-> flutter pub get Resolving dependencies... Because flutter_app depends on yumemi_lints ^2.2.0 and no versions of yumemi_lints match >2.2.0 <3.0.0, yumemi_lints 2.2.0 is required. And because yumemi_lints 2.2.0 depends on file ^6.1.4 and melos 7.0.0-dev.3 depends on file ^7.0.0, melos 7.0.0-dev.3 is forbidden. So, because no versions of melos match >7.0.0-dev.3 <8.0.0 and template_workspace depends on melos ^7.0.0-dev.3, version solving failed. Failed to update packages.

tatsutakein · 2025-01-13T11:40:16Z

pubspec.yaml

+
+dependency_overrides:
+  file: ^7.0.0
+  path: ^1.9.1


パッチバージョンのため問題なさそうだと思っています。

tatsutakein · 2025-01-13T11:44:30Z

.gitignore

+/.flutter-plugins
+/.flutter-plugins-dependencies


melos bs 実行時に、実態として flutter pub get が実行されてファイルが生成されるため gitignore に追加します。

❯ melos bs melos bootstrap └> /Users/xxxxx/flutter-mobile-project-template Updating common dependencies in workspace packages... > SUCCESS Running "/Users/xxxxx/flutter-mobile-project-template/.fvm/flutter_sdk/bin/flutter pub get" in workspace... [template_workspace]: Resolving dependencies... [template_workspace]: Downloading packages... [template_workspace]: _fe_analyzer_shared 76.0.0 (78.0.0 available) [template_workspace]: analyzer 6.11.0 (7.1.0 available) [template_workspace]: analyzer_plugin 0.11.3 (0.12.0 available) [template_workspace]: archive 3.6.1 (4.0.2 available) [template_workspace]: async 2.11.0 (2.12.0 available) [template_workspace]: boolean_selector 2.1.1 (2.1.2 available) [template_workspace]: characters 1.3.0 (1.4.0 available) [template_workspace]: clock 1.1.1 (1.1.2 available) [template_workspace]: collection 1.19.0 (1.19.1 available) [template_workspace]: custom_lint 0.7.0 (0.7.1 available) [template_workspace]: custom_lint_builder 0.7.0 (0.7.1 available) [template_workspace]: custom_lint_core 0.7.0 (0.7.1 available) [template_workspace]: custom_lint_visitor 1.0.0+6.11.0 (1.0.0+7.1.0 available) [template_workspace]: dart_style 2.3.7 (3.0.1 available) [template_workspace]: fake_async 1.3.1 (1.3.2 available) [template_workspace]: ! file 7.0.1 (overridden) [template_workspace]: freezed 2.5.7 (2.5.8 available) [template_workspace]: image 4.3.0 (4.5.2 available) [template_workspace]: intl 0.19.0 (0.20.1 available) [template_workspace]: json_serializable 6.9.0 (6.9.2 available) [template_workspace]: leak_tracker 10.0.7 (10.0.8 available) [template_workspace]: leak_tracker_flutter_testing 3.0.8 (3.0.9 available) [template_workspace]: matcher 0.12.16+1 (0.12.17 available) [template_workspace]: material_color_utilities 0.11.1 (0.12.0 available) [template_workspace]: meta 1.15.0 (1.16.0 available) [template_workspace]: ! path 1.9.1 (overridden) [template_workspace]: petitparser 6.0.2 (6.1.0 available) [template_workspace]: riverpod_analyzer_utils 0.5.8 (0.5.9 available) [template_workspace]: riverpod_generator 2.6.3 (2.6.4 available) [template_workspace]: riverpod_lint 2.6.3 (2.6.4 available) [template_workspace]: source_gen 1.5.0 (2.0.0 available) [template_workspace]: source_span 1.10.0 (1.10.1 available) [template_workspace]: stack_trace 1.12.0 (1.12.1 available) [template_workspace]: stream_channel 2.1.2 (2.1.4 available) [template_workspace]: string_scanner 1.3.0 (1.4.1 available) [template_workspace]: term_glyph 1.2.1 (1.2.2 available) [template_workspace]: test 1.25.8 (1.25.14 available) [template_workspace]: test_api 0.7.3 (0.7.4 available) [template_workspace]: test_core 0.6.5 (0.6.8 available) [template_workspace]: theme_extensions_builder 3.1.0 (4.0.1 available) [template_workspace]: theme_extensions_builder_annotation 3.1.0 (4.0.1 available) [template_workspace]: vm_service 14.3.0 (15.0.0 available) [template_workspace]: Got dependencies! [template_workspace]: 40 packages have newer versions incompatible with dependency constraints. [template_workspace]: Try `flutter pub outdated` for more information. > SUCCESS -> 15 packages bootstrapped

tatsutakein · 2025-01-13T11:46:49Z

.gitignore

-# Melos related
-/pubspec_overrides.yaml
-


pubspec_overrides.yaml ファイルは生成されなくなるため gitignore からも削除。

tatsutakein · 2025-01-13T11:49:06Z

pubspec.yaml

+melos:
+  sdkPath: .fvm/flutter_sdk
+
+  ide:
+    intellij:
+      # Disabling IntelliJ's automatic configuration.
+      # This is to support different build configurations for environments.
+      enabled: false
+


マイグレーションガイドに則り melos.yaml の内容をルートの pubspec.yaml に移動しています。

https://github.com/invertase/melos?tab=readme-ov-file#migrate-to-melos-7xx

以下の項目は移さずにいます。

name

packages

tatsutakein · 2025-01-13T11:54:42Z

apps/catalog/macos/Flutter/Flutter-Debug.xcconfig

この PR の本質ではない部分で申し訳ないですが、ビルド検証時に生成されたためコミットしています 🙏

github-actions · 2025-01-13T11:58:44Z

Ready for review 🚀

tatsutakein · 2025-01-15T06:29:39Z

apps/app/.gitignore

-
-# Melos related
-/pubspec_overrides.yaml


c0697a9 の際に対応漏れしていた部分を追加でコミットしています。

blendthink · 2025-01-17T09:07:28Z

@tatsutakein
すみません、、他のプルリクエストから先にマージしてしまってコンフリクトが発生してしまったので、ご対応お願いいたします 🙏

ただ、もうコンフリクト発生してしまったので #365 までマージしてからご対応していただけますでしょうか 🙇 🙇 🙇

tatsutakein

@blendthink
f6ec41f にてコンフリクト解消いたしました！
お手数ですが、再度ご確認いただけますでしょうか 🙏

実態は 6 files です。

tatsutakein · 2025-01-17T13:10:59Z

apps/catalog/.gitignore

+
+# Ignore non-CI golden files and failures
+test/**/goldens/**/*.*
+test/**/failures/**/*.*


コンフリクト解消

tatsutakein · 2025-01-17T13:12:38Z

apps/catalog/pubspec.yaml

 resolution: workspace

 dependencies:
+  alchemist: ^0.11.0


コンフリクト解消

tatsutakein · 2025-01-17T13:14:29Z

packages/samples/github_repository/pubspec.yaml

+  utils_pagination:
+    path: ../../utils/pagination


コンフリクト解消

tatsutakein · 2025-01-17T13:15:18Z

packages/utils/pagination/pubspec.yaml

+
+resolution: workspace


コンフリクト解消

tatsutakein · 2025-01-17T13:21:00Z

pubspec.yaml

melos.yaml も含めたコンフリクト解消

blendthink

@tatsutakein

手順通りに確認したところ、いくつか差分が発生してしまっているため、ご確認お願いできますでしょうか 🙏

blendthink · 2025-02-11T23:17:37Z

@blendthink が引き継ぐことになったためアサイン更新しました 💪

github-actions bot assigned tatsutakein Jan 13, 2025

tatsutakein added 2 commits January 13, 2025 20:00

fix(deps): theme_extensions_builder 4.0.0-> 3.1.0

f531fb7

build: パッケージ依存管理部分を melos -> pub workspaces へ移行

3d46157

tatsutakein force-pushed the feature/GH-362 branch from c0bf7fc to 8252547 Compare January 13, 2025 11:00

build: melos ^6.3.0 -> ^7.0.0-dev.3

c0697a9

tatsutakein force-pushed the feature/GH-362 branch from 8252547 to c0697a9 Compare January 13, 2025 11:15

build(catalog): macos 向けビルド時の生成ファイルを追加

ffe0751

tatsutakein commented Jan 13, 2025

View reviewed changes

tatsutakein marked this pull request as ready for review January 13, 2025 11:58

yumemi-team-review-requester bot requested review from a team and blendthink and removed request for a team January 13, 2025 11:58

chore: 生成されなくなった pubspec_overrides.yaml を gitignore の対象から削除

b5cb2d3

tatsutakein commented Jan 15, 2025

View reviewed changes

github-actions bot added the @packages/utils/pagination packages utils pagination package label Jan 17, 2025

tatsutakein force-pushed the feature/GH-362 branch 2 times, most recently from 3336b8a to d663bb7 Compare January 17, 2025 12:02

Merge branch 'refs/heads/main' into feature/GH-362

f6ec41f

tatsutakein force-pushed the feature/GH-362 branch from d663bb7 to f6ec41f Compare January 17, 2025 12:13

tatsutakein enabled auto-merge January 17, 2025 12:18

tatsutakein commented Jan 17, 2025

View reviewed changes

blendthink reviewed Jan 20, 2025

View reviewed changes

blendthink assigned blendthink and unassigned tatsutakein Feb 11, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

build: パッケージ依存管理部分を melos -> pub workspaces へ移行 #371

build: パッケージ依存管理部分を melos -> pub workspaces へ移行 #371

tatsutakein commented Jan 13, 2025 •

edited

Loading

github-actions bot commented Jan 13, 2025 •

edited

Loading

tatsutakein Jan 13, 2025

tatsutakein Jan 13, 2025

tatsutakein Jan 13, 2025

tatsutakein Jan 13, 2025 •

edited

Loading

tatsutakein Jan 13, 2025

tatsutakein Jan 13, 2025 •

edited

Loading

tatsutakein Jan 13, 2025

github-actions bot commented Jan 13, 2025

tatsutakein Jan 15, 2025

blendthink commented Jan 17, 2025

tatsutakein left a comment

tatsutakein Jan 17, 2025

tatsutakein Jan 17, 2025

tatsutakein Jan 17, 2025

tatsutakein Jan 17, 2025

tatsutakein Jan 17, 2025

blendthink left a comment

blendthink commented Feb 11, 2025

		/.flutter-plugins
		/.flutter-plugins-dependencies

build: パッケージ依存管理部分を melos -> pub workspaces へ移行 #371

Are you sure you want to change the base?

build: パッケージ依存管理部分を melos -> pub workspaces へ移行 #371

Conversation

tatsutakein commented Jan 13, 2025 • edited Loading

概要

やったこと

レビュー観点

レビューレベル

レビュー優先度

画像 / 動画

確認したこと

動作確認手順

備考

github-actions bot commented Jan 13, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tatsutakein Jan 13, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tatsutakein Jan 13, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Jan 13, 2025

Choose a reason for hiding this comment

blendthink commented Jan 17, 2025

tatsutakein left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

blendthink left a comment

Choose a reason for hiding this comment

blendthink commented Feb 11, 2025

tatsutakein commented Jan 13, 2025 •

edited

Loading

github-actions bot commented Jan 13, 2025 •

edited

Loading

tatsutakein Jan 13, 2025 •

edited

Loading

tatsutakein Jan 13, 2025 •

edited

Loading