Skip to content

chore: 自動生成後にフォーマットを実行しなくてよくする#576

Merged
blendthink merged 4 commits into
mainfrom
refactor/GH-529
Sep 1, 2025
Merged

chore: 自動生成後にフォーマットを実行しなくてよくする#576
blendthink merged 4 commits into
mainfrom
refactor/GH-529

Conversation

@blendthink
Copy link
Copy Markdown
Contributor

概要

close: #529

Dart 3.8更新によるフォーマット問題を解決するため、自動生成後にフォーマットを実行しないようにしました。
代わりに、専用のformatコマンドとformat:checkコマンドを追加し、フォーマットとコード生成を独立して実行できるようにしました。

レビュー観点

  • melosコマンドの設定変更が適切か
  • 新しく追加されたformatコマンドの設定が適切か
  • 既存のgenコマンドからdart formatが適切に削除されているか
  • 自動生成ファイルのフォーマット問題が解決されているか

レビューレベル

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

レビュー優先度

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

画像 / 動画

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

確認したこと

  • melosコマンドの設定が正しく更新されていること
  • 新しく追加されたformatコマンドが適切に設定されていること
  • 既存のgenコマンドからdart formatが削除されていること
  • 自動生成ファイルのフォーマット問題が解決されていること

動作確認手順

  1. melos formatを実行して、すべてのパッケージでフォーマットが実行されることを確認
  2. melos format:checkを実行して、フォーマットチェックが正常に動作することを確認
  3. melos gen:buildを実行して、build_runnerのみが実行されることを確認

備考

Dart 3.8更新により、自動生成パッケージでのフォーマット機能が問題を起こしていたため、一時的に自動生成後にフォーマットを実行していました。
PR #537でパッケージの更新が完了し、Dart 3.8でのフォーマット問題は再現できなくなったため、この変更により、フォーマットとコード生成を独立して実行できるようになり、CIパイプラインでの問題が解決されます。

@yumemi-team-review-requester yumemi-team-review-requester Bot requested review from a team, den0206 and morikann and removed request for a team August 28, 2025 11:41
@blendthink blendthink changed the title GH-529: 自動生成後にフォーマットを実行しなくてよくする chore: 自動生成後にフォーマットを実行しなくてよくする Aug 28, 2025
@github-actions
Copy link
Copy Markdown

Ready for review 🚀

@blendthink blendthink requested review from trm11tkr and removed request for den0206 August 28, 2025 11:41
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.

一点コメントさせていただきましたのでご確認よろしくお願いします🙏

Comment thread pubspec.yaml
@blendthink blendthink requested a review from trm11tkr September 1, 2025 03:58
Base automatically changed from refactor/GH-568 to main September 1, 2025 04:08
@blendthink blendthink enabled auto-merge September 1, 2025 04:09
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 1, 2025
Merged via the queue into main with commit 3a249c1 Sep 1, 2025
16 checks passed
@blendthink blendthink deleted the refactor/GH-529 branch September 1, 2025 04:33
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.

[Improve]: 自動生成後にフォーマットを実行しなくてよくする

2 participants