Skip to content

Commit b2fb991

Browse files
authored
Refactor workflow dependencies to remove 'common'
1 parent 6c5ed51 commit b2fb991

1 file changed

Lines changed: 15 additions & 9 deletions

File tree

.github/workflows/pull-request.yml

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,13 @@ jobs:
2323
code: ${{ steps.filter.outputs.code }}
2424
steps:
2525
- uses: actions/checkout@v6
26+
with:
27+
fetch-depth: 0
2628

2729
- id: filter
2830
uses: dorny/paths-filter@v3
2931
with:
32+
base: ${{ github.event.pull_request.base.sha }}
3033
filters: |
3134
code:
3235
- '**.dart'
@@ -39,6 +42,8 @@ jobs:
3942
4043
common:
4144
name: Common Build
45+
needs: changes
46+
if: needs.changes.outputs.code == 'true'
4247
runs-on: ubuntu-latest
4348
steps:
4449
- uses: actions/checkout@v6
@@ -58,7 +63,7 @@ jobs:
5863

5964
android:
6065
name: Android Flutter Build
61-
needs: [common, changes]
66+
needs: changes
6267
runs-on: ubuntu-latest
6368
steps:
6469
- uses: actions/checkout@v6
@@ -73,10 +78,11 @@ jobs:
7378

7479
ios:
7580
name: iOS Flutter Build
76-
needs: [common, changes]
81+
needs: changes
7782
runs-on: macos-latest
7883
steps:
7984
- name: Set up Xcode
85+
if: needs.changes.outputs.code == 'true'
8086
uses: maxim-lobanov/setup-xcode@v1.7.0
8187
with:
8288
xcode-version: '16.4.0'
@@ -93,7 +99,7 @@ jobs:
9399

94100
screenshots-android:
95101
name: Screenshots (Android)
96-
needs: [changes]
102+
needs: changes
97103
if: needs.changes.outputs.code == 'true'
98104
runs-on: ubuntu-latest
99105
timeout-minutes: 30
@@ -108,7 +114,7 @@ jobs:
108114

109115
screenshots-iphone:
110116
name: Screenshots (iPhone)
111-
needs: [changes]
117+
needs: changes
112118
if: needs.changes.outputs.code == 'true'
113119
runs-on: macos-latest
114120
timeout-minutes: 30
@@ -127,7 +133,7 @@ jobs:
127133

128134
screenshots-ipad:
129135
name: Screenshots (iPad)
130-
needs: [changes]
136+
needs: changes
131137
if: needs.changes.outputs.code == 'true'
132138
runs-on: macos-latest
133139
timeout-minutes: 30
@@ -146,7 +152,7 @@ jobs:
146152

147153
windows:
148154
name: Windows Flutter Build
149-
needs: [common, changes]
155+
needs: changes
150156
if: needs.changes.outputs.code == 'true'
151157
runs-on: windows-latest
152158
steps:
@@ -157,7 +163,7 @@ jobs:
157163

158164
linux:
159165
name: Linux Flutter Build
160-
needs: [common, changes]
166+
needs: changes
161167
if: needs.changes.outputs.code == 'true'
162168
runs-on: ubuntu-latest
163169
steps:
@@ -168,7 +174,7 @@ jobs:
168174

169175
linux-arm64:
170176
name: Linux ARM64 Flutter Build
171-
needs: [common, changes]
177+
needs: changes
172178
if: needs.changes.outputs.code == 'true'
173179
runs-on: ubuntu-24.04-arm
174180
steps:
@@ -179,7 +185,7 @@ jobs:
179185

180186
macos:
181187
name: macOS Flutter Build
182-
needs: [common, changes]
188+
needs: changes
183189
if: needs.changes.outputs.code == 'true'
184190
runs-on: macos-latest
185191
steps:

0 commit comments

Comments
 (0)