Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Ruby 3.4 preview 2 to the CI #1340

Merged
merged 5 commits into from
Dec 5, 2024
Merged

Add Ruby 3.4 preview 2 to the CI #1340

merged 5 commits into from
Dec 5, 2024

Conversation

tombruijn
Copy link
Member

@tombruijn tombruijn commented Dec 2, 2024

Add Ruby 3.4 preview 2 to the CI

Test against the latest prerelease to test if any issues arise.

[skip changeset]

Update Sidekiq probe config logging

In Ruby 3.4 the internal representation of Hashes has changed when called with to_s/inspect. Instead of {:abc => :def} it is now {abc: :def}.

Use own config formatter so we are not relying on the Ruby format and tests don't break over different Ruby versions.

Update Hash representation in Strings

In Ruby 3.4 the internal representation of Hashes has changed when called with to_s/inspect. Instead of {:abc => :def} it is now {abc: :def}.

Update the Data struct spec to match both formats. We don't really rely on the String representation of Data structs so it's fine.

For the ElasticSearch event formatter it does impact how we detect unique events. It will no longer match the event between Ruby 3.3 and Ruby 3.4 as the same event, meaning event metrics will be restarted after upgrading to Ruby 3.4. This is already what happens upgrading to Ruby 3.4 with published versions of the Ruby gem. And adding our own formatter for arbitrary search parameters seems brittle. We'd have to keep the Ruby format prior to Ruby 3.4 too.

Update gitignore for macOS Ruby 3.4 artifacts

We generate a bunch more artifacts on Ruby 3.4 on macOS. Git ignore them so we don't accidentally commit them.

Add required gems for tests on Ruby 3.4

Ruby 3.4 removed a couple of standard gems. Add them manually so the tests work.

@tombruijn tombruijn added the chore label Dec 2, 2024
@tombruijn tombruijn self-assigned this Dec 2, 2024
@backlog-helper
Copy link

backlog-helper bot commented Dec 2, 2024

✔️ All good!

New issue guide | Backlog management | Rules | Feedback

@tombruijn tombruijn marked this pull request as draft December 2, 2024 14:37
@tombruijn tombruijn force-pushed the ruby-3.4.0-preview2 branch 2 times, most recently from 8bf26e0 to 6fb18f8 Compare December 2, 2024 14:49
@tombruijn tombruijn marked this pull request as ready for review December 2, 2024 15:05
@tombruijn tombruijn force-pushed the ruby-3.4.0-preview2 branch from edf8329 to bb4b5eb Compare December 3, 2024 08:28
Test against the latest prerelease to test if any issues arise.

[skip changeset]
[skip review]
In Ruby 3.4 the internal representation of Hashes has changed when
called with `to_s`/`inspect`. Instead of `{:abc => :def}` it is now
`{abc: :def}`.

Use own config formatter so we are not relying on the Ruby format and
tests don't break over different Ruby versions.
In Ruby 3.4 the internal representation of Hashes has changed when
called with `to_s`/`inspect`. Instead of `{:abc => :def}` it is now
`{abc: :def}`.

Update the Data struct spec to match both formats. We don't really rely
on the String representation of Data structs so it's fine.

For the ElasticSearch event formatter it does impact how we detect
unique events. It will no longer match the event between Ruby 3.3 and
Ruby 3.4 as the same event, meaning event metrics will be restarted
after upgrading to Ruby 3.4. This is already what happens upgrading to
Ruby 3.4 with published versions of the Ruby gem. And adding our own
formatter for arbitrary search parameters seems brittle. We'd have to
keep the Ruby format prior to Ruby 3.4 too.
We generate a bunch more artifacts on Ruby 3.4 on macOS. Git ignore them
so we don't accidentally commit them.
Ruby 3.4 removed a couple of standard gems. Add them manually so the
tests work.
@tombruijn tombruijn force-pushed the ruby-3.4.0-preview2 branch from bb4b5eb to bfc2b5d Compare December 3, 2024 08:43
@backlog-helper
Copy link

backlog-helper bot commented Dec 5, 2024


This is a message from the daily scheduled checks.

New issue guide | Backlog management | Rules | Feedback

@tombruijn tombruijn merged commit 4b3039e into main Dec 5, 2024
151 checks passed
@tombruijn tombruijn deleted the ruby-3.4.0-preview2 branch December 5, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants