Skip to content

Add Ruby 3.1 to CI#116

Merged
Morriar merged 3 commits intoShopify:mainfrom
petergoldstein:feature/add_ruby_3_1_to_ci
Jan 31, 2022
Merged

Add Ruby 3.1 to CI#116
Morriar merged 3 commits intoShopify:mainfrom
petergoldstein:feature/add_ruby_3_1_to_ci

Conversation

@petergoldstein
Copy link
Contributor

This PR adds Ruby 3.1 to the CI matrix and shifts Ruby 3.0 out of experimental.

It also addresses some syntax issues in the CI config:

  1. 3.0 is now wrapped in quotes so that it is not truncated to 3. If unquoted, a 3.0 causes the latest Ruby 3 (at this point Ruby 3.1.0) to be loaded. To ensure a Ruby 3.0.x is loaded this must be quoted
  2. Removed the array in the os entry in the include. This should be a single element, not an array.

I'm seeing flaky failures on one individual spec, shifting between Ruby versions with different runs.

experimental: true

env:
RUBYOPT: "--disable-error_highlight"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this because I misidentified the DidYouMean failures in the Ruby 3.1 run as being a result of the error highlighting which is on by default in Ruby 3.1. It's not - it looks like some other issue, either in the Ruby itself or its supporting infrastructure. Removed.

@Morriar
Copy link
Contributor

Morriar commented Jan 31, 2022

Indeed there is something flaky with the run_test. I'll merge this and investigate (issue #117).

@Morriar Morriar merged commit 96634e2 into Shopify:main Jan 31, 2022
@shopify-shipit shopify-shipit bot temporarily deployed to production March 22, 2022 15:02 Inactive
@Morriar Morriar added the chore Chore task label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Chore task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants