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

Update to Ruby 3.4.1 #11856

Merged
merged 6 commits into from
Feb 7, 2025
Merged

Update to Ruby 3.4.1 #11856

merged 6 commits into from
Feb 7, 2025

Conversation

mitchellhenke
Copy link
Contributor

🛠 Summary of changes

Ruby 3.4 was released a little over a month ago. This PR upgrades us to use the most recent version.

@mitchellhenke mitchellhenke requested a review from a team February 7, 2025 19:55
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/ruby-3.4.1 branch 2 times, most recently from 832e52c to 60ac410 Compare February 7, 2025 20:08
.rubocop.yml Outdated Show resolved Hide resolved
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/ruby-3.4.1 branch from 60ac410 to 9670a4c Compare February 7, 2025 20:19
mitchellhenke and others added 3 commits February 7, 2025 15:25
changelog: Internal, Maintenance, Update to Ruby 3.4.1
Co-authored-by: Andrew Duthie <[email protected]>
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/ruby-3.4.1 branch from cb978b8 to 9acb00e Compare February 7, 2025 21:31
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/ruby-3.4.1 branch from 9151a39 to f45a475 Compare February 7, 2025 21:51
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/ruby-3.4.1 branch from f45a475 to 38bc641 Compare February 7, 2025 21:58
@@ -84,7 +84,7 @@ def track_event(event, original_attributes = {})
method_name = caller
.grep(/analytics_events\.rb/)
&.first
&.match(/:in `(?<method_name>[^']+)'/)
&.match(/:in '[\w\d]*#(?<method_name>[^']+)'/)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The caller method returns a different format in 3.4:

- "identity-idp/app/services/analytics_events.rb:4409:in `idv_phone_confirmation_otp_submitted'"
+ "identity-idp/app/services/analytics_events.rb:4409:in 'AnalyticsEvents#idv_phone_confirmation_otp_submitted'"

@mitchellhenke mitchellhenke merged commit 4977a5d into main Feb 7, 2025
2 checks passed
@mitchellhenke mitchellhenke deleted the mitchellhenke/ruby-3.4.1 branch February 7, 2025 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants