Add TruffleRuby in CI #305
Conversation
* We need to use truffleruby-head for now because of oracle/truffleruby#3683
|
Awesome, thanks @eregon! |
| @@ -16,7 +16,11 @@ jobs: | |||
| ruby-version: | |||
| - "3.2" | |||
| - "3.3" | |||
| - "truffleruby-head" | |||
There was a problem hiding this comment.
We could now add or switch to truffleruby-24.1.2 in CI, e.g. with "truffleruby" (latest release), WDYT @fxn ?
There was a problem hiding this comment.
Hey! Thanks for addressing that!
I created #311 but some tests are failing, maybe give it a time for packages to be available?
There was a problem hiding this comment.
Right, https://github.com/ruby/setup-ruby/actions/runs/12989568868 was 33min ago and your PR 34min ago so probably just a bit too fast, could you rerun the CI on that PR?
There was a problem hiding this comment.
Greener! There's just one failure on Windows, error says is is unsupported. Should we remove that pair from the matrix?
There was a problem hiding this comment.
Yes, the error is accurate, so that combination should be excluded
Now all tests pass, except oracle/truffleruby#2431 for well-known reasons.
I would like to add TruffleRuby in CI so it's easy to notice any potential issue going forward.