Skip to content

Commit dbca138

Browse files
grdsdevclaude
andcommitted
refactor(ci): always use --no-flutter in reusable workflow
Since all packages using the reusable dart-package-test workflow are Dart-only packages (no Flutter SDK), the --no-flutter flag is always needed. Removed the melos-no-flutter input parameter and simplified the bootstrap step to always use --no-flutter. This simplifies the workflow configuration and removes unnecessary conditional logic, as all 7 packages (gotrue, storage_client, realtime_client, supabase, functions_client, postgrest, yet_another_json_isolate) require this flag. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 861d9d3 commit dbca138

File tree

8 files changed

+1
-17
lines changed

8 files changed

+1
-17
lines changed

.github/workflows/dart-package-test.yml

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ on:
2525
type: number
2626
default: 0
2727
description: 'Test concurrency level (0 means no flag, 1 means --concurrency=1)'
28-
melos-no-flutter:
29-
required: false
30-
type: boolean
31-
default: false
32-
description: 'Whether to use --no-flutter flag with melos bootstrap'
3328

3429
jobs:
3530
test:
@@ -71,11 +66,7 @@ jobs:
7166
run: |
7267
cd ../../
7368
dart pub global activate melos
74-
if [ "${{ inputs.melos-no-flutter }}" == "true" ]; then
75-
melos bootstrap --no-flutter
76-
else
77-
melos bootstrap
78-
fi
69+
melos bootstrap --no-flutter
7970
8071
- name: dartfmt
8172
if: ${{ matrix.sdk == 'stable'}}

.github/workflows/functions_client.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,4 +30,3 @@ jobs:
3030
with:
3131
package-name: functions_client
3232
working-directory: packages/functions_client
33-
melos-no-flutter: true

.github/workflows/gotrue.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,3 @@ jobs:
3131
needs-docker: true
3232
docker-compose-dir: infra/gotrue
3333
test-concurrency: 1
34-
melos-no-flutter: true

.github/workflows/postgrest.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,4 +33,3 @@ jobs:
3333
needs-docker: true
3434
docker-compose-dir: infra/postgrest
3535
test-concurrency: 1
36-
melos-no-flutter: true

.github/workflows/realtime_client.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,4 +29,3 @@ jobs:
2929
package-name: realtime_client
3030
working-directory: packages/realtime_client
3131
test-concurrency: 1
32-
melos-no-flutter: true

.github/workflows/storage_client.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,3 @@ jobs:
3131
needs-docker: true
3232
docker-compose-dir: infra/storage_client
3333
test-concurrency: 1
34-
melos-no-flutter: true

.github/workflows/supabase.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,4 +39,3 @@ jobs:
3939
package-name: supabase
4040
working-directory: packages/supabase
4141
test-concurrency: 1
42-
melos-no-flutter: true

.github/workflows/yet_another_json_isolate.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,3 @@ jobs:
2828
with:
2929
package-name: yet_another_json_isolate
3030
working-directory: packages/yet_another_json_isolate
31-
melos-no-flutter: true

0 commit comments

Comments
 (0)