Skip to content

Commit 3a249c1

Browse files
authored
Merge pull request #576 from yumemi-inc/refactor/GH-529
chore: 自動生成後にフォーマットを実行しなくてよくする
2 parents 10390db + 8d55fb8 commit 3a249c1

3 files changed

Lines changed: 35 additions & 8 deletions

File tree

.github/workflows/check-pr.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ jobs:
8585
fatal-infos: true
8686

8787
- name: Check format
88-
run: melos format --output none --set-exit-if-changed
88+
run: melos run format:check
8989

9090
- name: Check for circular dependencies
9191
run: melos list --cycles

.github/workflows/merge-queue.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ jobs:
8484
fatal-infos: true
8585

8686
- name: Check format
87-
run: melos format --output none --set-exit-if-changed
87+
run: melos run format:check
8888

8989
- name: Check for circular dependencies
9090
run: melos list --cycles

pubspec.yaml

Lines changed: 33 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,33 @@ melos:
102102
dirExists: lib
103103
dependsOn: custom_lint
104104

105+
format:
106+
description: |
107+
すべてのパッケージに `dart format` を実行します。
108+
自動生成されたコードは対象外になるように設定しています。
109+
exec: >-
110+
find lib -name "*.dart"
111+
-not -path "*/gen/*"
112+
-not -name "*.g.dart"
113+
-not -name "*.freezed.dart"
114+
-not -name "*.gen.dart"
115+
-not -name "*.g.theme.dart" | xargs dart format
116+
packageFilters:
117+
dirExists: lib
118+
format:check:
119+
description: |
120+
すべてのパッケージに `dart format --output=none --set-exit-if-changed` を実行します。
121+
自動生成されたコードは対象外になるように設定しています。
122+
exec: >-
123+
find lib -name "*.dart"
124+
-not -path "*/gen/*"
125+
-not -name "*.g.dart"
126+
-not -name "*.freezed.dart"
127+
-not -name "*.gen.dart"
128+
-not -name "*.g.theme.dart" | xargs dart format --output=none --set-exit-if-changed
129+
packageFilters:
130+
dirExists: lib
131+
105132
gen:
106133
description: build_runner と l10n の生成コマンドを実行します。
107134
steps:
@@ -114,29 +141,29 @@ melos:
114141
- gen:build:diff:main
115142
gen:build:
116143
description: build_runner を使用してコードを生成します。
117-
run: dart run build_runner build -d && dart format .
144+
run: dart run build_runner build -d
118145
exec:
119146
orderDependents: true
120147
packageFilters:
121148
dependsOn: build_runner
122149
gen:l10n:
123150
description: 多言語対応のためのローカライゼーションファイルを生成します。
124-
run: flutter gen-l10n && dart format .
151+
run: flutter gen-l10n
125152
exec:
126153
orderDependents: true
127154
packageFilters:
128155
dependsOn: flutter_localizations
129156
gen:build:diff:main:
130157
description: main ブランチと差分のあるパッケージのみ build_runner を使用してコードを生成します。
131-
run: dart run build_runner build -d && dart format .
158+
run: dart run build_runner build -d
132159
exec:
133160
orderDependents: true
134161
packageFilters:
135162
dependsOn: build_runner
136163
diff: origin/main...HEAD
137164
gen:l10n:diff:main:
138165
description: main ブランチと差分のあるパッケージのみ 多言語対応のためのローカライゼーションファイルを生成します。
139-
run: flutter gen-l10n && dart format .
166+
run: flutter gen-l10n
140167
exec:
141168
orderDependents: true
142169
packageFilters:
@@ -151,15 +178,15 @@ melos:
151178
- gen:l10n:diff:head
152179
gen:build:diff:head:
153180
description: 未コミットの差分パッケージのみ build_runner を使用してコードを生成します。
154-
run: dart run build_runner build -d && dart format .
181+
run: dart run build_runner build -d
155182
exec:
156183
orderDependents: true
157184
packageFilters:
158185
dependsOn: build_runner
159186
diff: ""
160187
gen:l10n:diff:head:
161188
description: 未コミットの差分パッケージのみ ローカライゼーションファイル を生成します。
162-
run: flutter gen-l10n && dart format .
189+
run: flutter gen-l10n
163190
exec:
164191
orderDependents: true
165192
packageFilters:

0 commit comments

Comments
 (0)