Skip to content

Commit a09adba

Browse files
committed
Tweak github actions to work around old kwargs syntax
rails7 stable appears to still use a ruby-2.7-- syntax that does not work with minitest mocks unless MT_KWARGS_HACK is set. I cannot see where rails CI does this but I can see on head of Rails development they corrected all the tests we use to 'expect' explicit kwargs vs stuffing them onto the args parameter.
1 parent 05a9954 commit a09adba

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

.github/workflows/ruby.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ jobs:
4343
PREPARED_STATEMENTS: ${{ matrix.prepared_statements }}
4444
DRIVER: ${{ matrix.driver }}
4545
JRUBY_OPTS: "-J-Xms64M -J-Xmx1024M"
46+
MT_KWARGS_HACK: 1
4647

4748
steps:
4849
- uses: actions/checkout@v3
@@ -100,6 +101,8 @@ jobs:
100101
PGHOST: localhost
101102
PGPORT: 5432
102103
PGUSER: postgres
104+
MT_KWARGS_HACK: 1
105+
103106

104107
steps:
105108
- uses: actions/checkout@v3
@@ -135,6 +138,7 @@ jobs:
135138
DB: ${{ matrix.db }}
136139
AR_VERSION: ${{ matrix.ar_version }}
137140
JRUBY_OPTS: "-J-Xms64M -J-Xmx1024M"
141+
MT_KWARGS_HACK: 1
138142

139143
steps:
140144
- uses: actions/checkout@v3
@@ -177,6 +181,7 @@ jobs:
177181
MY_USER: root
178182
MY_PASSWORD: root
179183
PREPARED_STATEMENTS: ${{ matrix.prepared_statements }}
184+
MT_KWARGS_HACK: 1
180185

181186
steps:
182187
- uses: actions/checkout@v3
@@ -228,6 +233,7 @@ jobs:
228233
PGHOST: localhost
229234
PGPORT: 5432
230235
PGUSER: postgres
236+
MT_KWARGS_HACK: 1
231237

232238
steps:
233239
- uses: actions/checkout@v3
@@ -257,6 +263,7 @@ jobs:
257263
env:
258264
DB: ${{ matrix.db }}
259265
JRUBY_OPTS: "-J-Xms64M -J-Xmx1024M"
266+
MT_KWARGS_HACK: 1
260267

261268
steps:
262269
- uses: actions/checkout@v3

0 commit comments

Comments
 (0)