Skip to content

Commit 1bd60c2

Browse files
authored
Merge pull request freqtrade#12554 from freqtrade/restructure_ci
restructure CI to reliably fail the build step if other ch…
2 parents c0a1911 + f4920f1 commit 1bd60c2

File tree

1 file changed

+15
-9
lines changed

1 file changed

+15
-9
lines changed

.github/workflows/ci.yml

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -275,10 +275,7 @@ jobs:
275275
# Notify only once - when CI completes (and after deploy) in case it's successful
276276
notify-complete:
277277
needs: [
278-
tests,
279-
docs-check,
280-
mypy-version-check,
281-
pre-commit,
278+
build,
282279
build-linux-online
283280
]
284281
runs-on: ubuntu-22.04
@@ -306,11 +303,23 @@ jobs:
306303
webhookUrl: ${{ secrets.DISCORD_WEBHOOK }}
307304

308305
build:
306+
if: always()
309307
name: "Build"
310-
needs: [ tests, docs-check, mypy-version-check, pre-commit ]
308+
needs: [
309+
tests,
310+
docs-check,
311+
mypy-version-check,
312+
pre-commit,
313+
]
311314
runs-on: ubuntu-22.04
312315

313316
steps:
317+
318+
- name: Decide whether the needed jobs succeeded or failed
319+
uses: re-actors/alls-green@05ac9388f0aebcb5727afa17fcccfecd6f8ec5fe # v1.2.2
320+
with:
321+
jobs: ${{ toJSON(needs) }}
322+
314323
- uses: actions/checkout@v5
315324
with:
316325
persist-credentials: false
@@ -405,10 +414,7 @@ jobs:
405414
docker-build:
406415
name: "Docker Build and Deploy"
407416
needs: [
408-
tests,
409-
docs-check,
410-
mypy-version-check,
411-
pre-commit
417+
build,
412418
]
413419
if: (github.event_name == 'push' || github.event_name == 'schedule' || github.event_name == 'release') && github.repository == 'freqtrade/freqtrade'
414420
uses: ./.github/workflows/docker-build.yml

0 commit comments

Comments
 (0)