Skip to content

Commit

Permalink
workflows: move tap syntax check to run on push
Browse files Browse the repository at this point in the history
  • Loading branch information
Bo98 committed Nov 24, 2023
1 parent 7ab2209 commit 0f7b1d1
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 7 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ jobs:

- run: brew test-bot --only-setup

- run: brew test-bot --only-tap-syntax

- name: Build Portable Ruby
run: |
mkdir -p bottle/
Expand Down
22 changes: 17 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -1,25 +1,37 @@
name: CI

on:
push:
branches: master
pull_request:

permissions:
contents: read

concurrency:
group: "${{ github.ref }}"
cancel-in-progress: true
cancel-in-progress: ${{ github.event_name == 'pull_request' }}

jobs:
tests:
syntax:
runs-on: ubuntu-latest
steps:
- name: Set up Homebrew
uses: Homebrew/actions/setup-homebrew@master

- run: brew test-bot --only-tap-syntax

build:
needs: syntax
if: github.event_name != 'push'
uses: ./.github/workflows/build.yml

# This job is used as a required status check, instead of requiring each build matrix element.
# When using ephemeral runners, the name of those runs change every time so they cannot be set as required.
conclusion:
needs: tests
needs: build
runs-on: ubuntu-latest
if: always()
if: github.event_name != 'push' && !cancelled()
steps:
- name: Result
run: ${{ needs.tests.result == 'success' }}
run: ${{ needs.build.result == 'success' }}

0 comments on commit 0f7b1d1

Please sign in to comment.