Skip to content

Commit

Permalink
Merge macOS CIs to reduce concurrency
Browse files Browse the repository at this point in the history
On GitHub Actions, the macOS runners seem much more expensive than
Ubuntu, and its limit is the most significant bottlenecks for our
CIs.  As the "check" tasks usually finish 3 or 4 times faster than
"test-bundler-parallel", it will be balanced by running all three
"check" tasks sequentially.
  • Loading branch information
nobu committed Oct 28, 2021
1 parent 3bf3697 commit 0c587e5
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 81 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ jobs:
test_task: [ "check", "test-bundler-parallel" ] #, "test-bundled-gems" ] matrix tests has unknown issues
fail-fast: false
env:
RUBY_TESTOPTS: '-q --tty=no'
GITPULLOPTIONS: --no-tags origin ${{github.ref}}
runs-on: ${{ matrix.os }}
steps:
Expand Down Expand Up @@ -52,8 +53,13 @@ jobs:
- run: make -s ${{ matrix.test_task }}
timeout-minutes: ${{ matrix.test_task == 'check' && 15 || 40 }}
env:
RUBY_TESTOPTS: "-q --tty=no"
TEST_BUNDLED_GEMS_ALLOW_FAILURES: "rexml"
- run: make -s check RUN_OPTS="--yjit"
timeout-minutes: 60
if: ${{ matrix.test_task == 'check' }}
- run: make -s check RUN_OPTS="--yjit --yjit-call-threshold=1 --yjit-max-versions=1"
timeout-minutes: 60
if: ${{ matrix.test_task == 'check' }}
- uses: k0kubun/action-slack@v2.0.0
with:
payload: |
Expand Down
80 changes: 0 additions & 80 deletions .github/workflows/yjit-macos.yml

This file was deleted.

0 comments on commit 0c587e5

Please sign in to comment.