Skip to content

Commit

Permalink
Add check step to ci
Browse files Browse the repository at this point in the history
  • Loading branch information
adriangb committed Oct 18, 2023
1 parent 51dc755 commit 53d3512
Show file tree
Hide file tree
Showing 3 changed files with 71 additions and 11 deletions.
28 changes: 24 additions & 4 deletions .github/workflows/python-json.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -296,10 +296,15 @@ jobs:
run: |
make lint
release:
name: Release
if: github.ref == format('refs/heads/{0}', github.event.repository.default_branch)
runs-on: ubuntu-latest
# https://github.com/marketplace/actions/alls-green
check: # This job does nothing and is only used for the branch protection
name: arrow-json Python Check
if: always()

outputs:
result: ${{ steps.all-green.outputs.result }}

needs:
- lint
- macos
Expand All @@ -309,6 +314,21 @@ jobs:
- musllinux
- musllinux-cross
- pypy
runs-on: ubuntu-latest

steps:
- name: Decide whether the needed jobs succeeded or failed
uses: re-actors/alls-green@release/v1
id: all-green
with:
jobs: ${{ toJSON(needs) }}

release:
name: Release
if: github.ref == format('refs/heads/{0}', github.event.repository.default_branch)
runs-on: ubuntu-latest
needs:
- check
steps:
- uses: actions/download-artifact@v2
with:
Expand Down
27 changes: 23 additions & 4 deletions .github/workflows/python.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -296,10 +296,14 @@ jobs:
run: |
make lint
release:
name: Release
if: github.ref == format('refs/heads/{0}', github.event.repository.default_branch)
runs-on: ubuntu-latest
# https://github.com/marketplace/actions/alls-green
check: # This job does nothing and is only used for the branch protection
name: pgpq Python Check
if: always()

outputs:
result: ${{ steps.all-green.outputs.result }}

needs:
- lint
- macos
Expand All @@ -309,6 +313,21 @@ jobs:
- musllinux
- musllinux-cross
- pypy
runs-on: ubuntu-latest

steps:
- name: Decide whether the needed jobs succeeded or failed
uses: re-actors/alls-green@release/v1
id: all-green
with:
jobs: ${{ toJSON(needs) }}

release:
name: Release
if: github.ref == format('refs/heads/{0}', github.event.repository.default_branch)
runs-on: ubuntu-latest
needs:
- check
steps:
- uses: actions/download-artifact@v2
with:
Expand Down
27 changes: 24 additions & 3 deletions .github/workflows/rust.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ on: [push, pull_request]
name: Rust CI

jobs:
check:
name: Check
cargo_check:
name: Cargo Check
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
Expand Down Expand Up @@ -63,12 +63,33 @@ jobs:
with:
command: test

# https://github.com/marketplace/actions/alls-green
check: # This job does nothing and is only used for the branch protection
name: pgpq Rust Check
if: always()

outputs:
result: ${{ steps.all-green.outputs.result }}

needs:
- cargo_check
- fmt
- clippy
- test
runs-on: ubuntu-latest

steps:
- name: Decide whether the needed jobs succeeded or failed
uses: re-actors/alls-green@release/v1
id: all-green
with:
jobs: ${{ toJSON(needs) }}

release:
name: Release
runs-on: ubuntu-latest
needs:
- check
- test
if: ${{ github.ref == 'refs/heads/main' }}
steps:
- uses: actions/checkout@v3
Expand Down

0 comments on commit 53d3512

Please sign in to comment.