Skip to content

Commit c7a5392

Browse files
committed
Move prism to the main workflow matrix
rubocop-ast depends on prism, so no need to add it to the gemfile anymore
1 parent 8f17849 commit c7a5392

File tree

1 file changed

+8
-22
lines changed

1 file changed

+8
-22
lines changed

.github/workflows/main.yml

Lines changed: 8 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,21 @@ jobs:
3535
- "3.4"
3636
- ruby-head
3737
- jruby-9.4
38-
name: "Ruby ${{ matrix.ruby }} - Spec"
38+
parser_engine:
39+
- parser_whitequark
40+
include:
41+
- ruby: "2.7" # Minimum version required for Prism to run.
42+
parser_engine: parser_prism
43+
name: "Ruby ${{ matrix.ruby }} - Spec (${{ matrix.parser_engine }})"
3944
steps:
4045
- uses: actions/checkout@v4
4146
- uses: ruby/setup-ruby@v1
4247
with:
4348
ruby-version: "${{ matrix.ruby }}"
4449
bundler-cache: true
4550
- run: NO_COVERAGE=true bundle exec rake spec
51+
env:
52+
PARSER_ENGINE: ${{ matrix.parser_engine }}
4653

4754
coverage:
4855
runs-on: ubuntu-latest
@@ -119,24 +126,3 @@ jobs:
119126
ruby-version: "3.4"
120127
bundler-cache: true
121128
- run: NO_COVERAGE=true bundle exec rake spec
122-
123-
prism:
124-
runs-on: ubuntu-latest
125-
name: Prism
126-
steps:
127-
- uses: actions/checkout@v4
128-
- name: Use prism parser
129-
run: |
130-
cat << EOF > Gemfile.local
131-
gem 'prism'
132-
EOF
133-
- name: set up Ruby
134-
uses: ruby/setup-ruby@v1
135-
with:
136-
# Specify the minimum Ruby version 2.7 required for Prism to run.
137-
ruby-version: "2.7"
138-
bundler-cache: true
139-
- env:
140-
NO_COVERAGE: true
141-
PARSER_ENGINE: parser_prism
142-
run: bundle exec rake spec

0 commit comments

Comments
 (0)