Skip to content

Commit

Permalink
ci: Build YJIT enabled matrix
Browse files Browse the repository at this point in the history
- Remove `doc-build-latest-with-yjit` since it's covered by matrix
- switch YJIT using `RUBY_YJIT_ENABLE`
  • Loading branch information
toshimaru committed Nov 24, 2024
1 parent c728139 commit 1f4f780
Showing 1 changed file with 11 additions and 19 deletions.
30 changes: 11 additions & 19 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,12 @@ on:
jobs:
jekyll-build:
runs-on: ubuntu-latest
name: Jekyll Build
name: Jekyll Build (YJIT_ENABLED ${{ matrix.yjit-enabled }})
strategy:
matrix:
yjit-enabled: [0, 1]
env:
RUBY_YJIT_ENABLE: ${{ matrix.yjit-enabled }}
steps:
- uses: actions/checkout@v4
- uses: ruby/setup-ruby@v1
Expand All @@ -21,25 +26,12 @@ jobs:

doc-build-latest:
runs-on: ubuntu-latest
name: Rails Doc Build (latest)
steps:
- uses: actions/checkout@v4
with:
submodules: true
- name: fetch Rails tags
run: cd rails && git fetch --depth=1 origin refs/tags/v7*:refs/tags/v7*
- uses: ruby/setup-ruby@v1
with:
ruby-version: 3.3
bundler-cache: true
- name: Doc Build
run: rake build

doc-build-latest-with-yjit:
runs-on: ubuntu-latest
name: Rails Doc Build (latest, with YJIT enabled)
name: Rails Doc Build (latest, YJIT_ENABLED ${{ matrix.yjit-enabled }})
strategy:
matrix:
yjit-enabled: [0, 1]
env:
RUBY_YJIT_ENABLE: true
RUBY_YJIT_ENABLE: ${{ matrix.yjit-enabled }}
steps:
- uses: actions/checkout@v4
with:
Expand Down

0 comments on commit 1f4f780

Please sign in to comment.