Skip to content

Commit 4088fd8

Browse files
authored
ci(shorebird): remove shard_runner package; now lives in _build_engine (#145)
The shard_runner package moved to shorebirdtech/_build_engine in shorebirdtech/_build_engine#209. Runner code lives next to the workflows that invoke it; runner changes no longer need a Flutter PR + engine bump round-trip to land + test. What stays in this repo (still read by the runner via --config-dir): - shorebird/ci/shards/*.json - shorebird/ci/compose.json - shorebird/ci/artifacts_manifest.template.yaml Also drops the 🧩 Shard Runner Tests job from shorebird_ci.yml and the matching entry from the `ci` aggregator's needs list — those tests now run in _build_engine's CI. Depends on shorebirdtech/_build_engine#209 landing first; otherwise in-flight engine builds against pre-bump SHAs would be unaffected (workflow points at _build_engine's runner) but anyone iterating locally on the runner would be confused by the deleted dir.
1 parent bf082fe commit 4088fd8

18 files changed

Lines changed: 0 additions & 2254 deletions

.github/workflows/shorebird_ci.yml

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -38,34 +38,6 @@ jobs:
3838
run: ../../bin/flutter test test/general.shard
3939
working-directory: packages/flutter_tools
4040

41-
shard-runner-tests:
42-
runs-on: ubuntu-latest
43-
44-
name: 🧩 Shard Runner Tests
45-
46-
defaults:
47-
run:
48-
working-directory: shorebird/ci/shard_runner
49-
50-
steps:
51-
- name: 📚 Git Checkout
52-
uses: actions/checkout@v4
53-
54-
- name: 🎯 Setup Dart
55-
uses: dart-lang/setup-dart@v1
56-
57-
- name: 📦 Install Dependencies
58-
run: dart pub get
59-
60-
- name: 🔍 Check Formatting
61-
run: dart format --output=none --set-exit-if-changed .
62-
63-
- name: 🔎 Analyze
64-
run: dart analyze --fatal-infos
65-
66-
- name: 🧪 Run Tests
67-
run: dart test
68-
6941
shorebird-android-tests:
7042
# Android tests run on Ubuntu (faster runners, no macOS needed)
7143
runs-on: ubuntu-latest
@@ -148,7 +120,6 @@ jobs:
148120
if: always()
149121
needs:
150122
- flutter-tools-tests
151-
- shard-runner-tests
152123
- shorebird-android-tests
153124
- shorebird-ios-tests
154125
runs-on: ubuntu-latest

shorebird/ci/shard_runner/.gitignore

Lines changed: 0 additions & 2 deletions
This file was deleted.

shorebird/ci/shard_runner/analysis_options.yaml

Lines changed: 0 additions & 7 deletions
This file was deleted.

shorebird/ci/shard_runner/bin/compare_buckets.dart

Lines changed: 0 additions & 158 deletions
This file was deleted.

shorebird/ci/shard_runner/bin/compose.dart

Lines changed: 0 additions & 104 deletions
This file was deleted.

0 commit comments

Comments
 (0)