Skip to content

feat(design_ui): assetsをresディレクトリ配下に移動#591

Merged
blendthink merged 2 commits into
mainfrom
improve/design-ui-move-assets-to-res
Sep 4, 2025
Merged

feat(design_ui): assetsをresディレクトリ配下に移動#591
blendthink merged 2 commits into
mainfrom
improve/design-ui-move-assets-to-res

Conversation

@blendthink
Copy link
Copy Markdown
Contributor

概要

packages/design_ui の assets を res 配下に移動しました。
プロジェクトのリソース管理規約に従って、すべてのリソースを res 配下で整理する対応です。

レビュー観点

  • アセットファイルの移動が正しく行われているか
  • pubspec.yaml のアセットパスが正しく更新されているか
  • 生成された assets.gen.dart が新しいパス構造を正しく反映しているか
  • 既存のアセット参照に影響がないか

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する

レビュー優先度

  • 数日以内で見てもらいたい 🐢

画像 / 動画

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

確認したこと

  • dart analyze でエラーがないこと
  • melos run gen:i18n が正常に実行されること
  • melos run gen:build でアセット生成が正常に実行されること
  • アセットファイルが res/assets/ に正しく移動されていること
  • pubspec.yaml のアセットパスが res/assets/ に更新されていること
  • 既存のアセット参照を新しいパス構造に更新したこと

動作確認手順

  1. melos bs を実行してワークスペースを更新
  2. dart analyze で静的解析を実行
  3. melos run gen:build でコード生成を実行
  4. アプリをビルドしてアセットが正しく読み込まれることを確認

備考

  • アセットファイルの移動により、生成される assets.gen.dart の構造が変更されています
  • 既存のアセット参照は Assets.res.assets.yumemiLogo の形式でアクセス可能です
  • この変更により、プロジェクト全体でリソース管理が統一されます

- packages/design_uiのassets/ディレクトリをres/assets/に移動
- pubspec.yamlのアセットパスをres/assets/に更新
- 生成されたassets.gen.dartを新しいパス構造に合わせて更新
- プロジェクトのリソース管理規約に合わせてres/配下でリソースを整理
- CommonAssets.yumemiLogo を CommonAssets.res.assets.yumemiLogo に変更
- 行の長さ制限に合わせてコードを整形
@github-actions github-actions Bot added the @packages/design_ui packages design_ui package label Sep 4, 2025
@yumemi-team-review-requester yumemi-team-review-requester Bot requested review from a team, karas1910 and trm11tkr and removed request for a team September 4, 2025 08:19
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 4, 2025

Ready for review 🚀

Copy link
Copy Markdown
Contributor

@trm11tkr trm11tkr 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 added this pull request to the merge queue Sep 4, 2025
Merged via the queue into main with commit 7e1c90c Sep 4, 2025
16 checks passed
@blendthink blendthink deleted the improve/design-ui-move-assets-to-res branch September 4, 2025 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@apps/app Application development @packages/design_ui packages design_ui package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improve]: packages/design_ui の assets を res 配下に移動

2 participants