Skip to content

Commit

Permalink
ci: only build images if everything passes
Browse files Browse the repository at this point in the history
  • Loading branch information
aldy505 committed Sep 8, 2024
1 parent 68180b1 commit e413622
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 8 deletions.
8 changes: 0 additions & 8 deletions .github/workflows/sdks-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@ jobs:

- name: Test
run: go test -v -coverprofile=coverage.out -covermode=atomic -race ./...
env:
PESTO_TOKEN: ${{ secrets.PESTO_TOKEN }}

- name: Codecov
uses: codecov/codecov-action@v3
Expand Down Expand Up @@ -63,8 +61,6 @@ jobs:

- name: Test
run: npx vitest run --coverage --threads=false
env:
PESTO_TOKEN: ${{ secrets.PESTO_TOKEN }}

- name: Codecov
uses: codecov/codecov-action@v3
Expand All @@ -91,8 +87,6 @@ jobs:

- name: Test
run: dotnet test Pesto.sln --collect "XPlat Code Coverage"
env:
PESTO_TOKEN: ${{ secrets.PESTO_TOKEN }}

- name: Codecov
uses: codecov/codecov-action@v3
Expand Down Expand Up @@ -131,8 +125,6 @@ jobs:

- name: Run Pytest
run: pytest . --cov-report term-missing --cov-report xml --cov pesto
env:
PESTO_TOKEN: ${{ secrets.PESTO_TOKEN }}

- name: Code Coverage Report
uses: codecov/codecov-action@v3
Expand Down
17 changes: 17 additions & 0 deletions .github/workflows/systems-nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,11 @@ jobs:
name: Build Database
runs-on: ubuntu-latest
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -166,6 +171,9 @@ jobs:
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -217,7 +225,10 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -259,7 +270,10 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 720
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -311,6 +325,9 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
Expand Down
12 changes: 12 additions & 0 deletions .github/workflows/systems-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,9 @@ jobs:
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -160,7 +163,10 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -201,7 +207,10 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 720
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
packages: write
Expand Down Expand Up @@ -242,6 +251,9 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 30
needs:
- auth
- rce
- landing
- registration
permissions:
contents: read
Expand Down

0 comments on commit e413622

Please sign in to comment.