-
Notifications
You must be signed in to change notification settings - Fork 11
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 Ruby to 3.3.4 #3519
Update Ruby to 3.3.4 #3519
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m unable to test this locally because we’re at the limit for the number of Mac devices for our apple development team. The same change worked fine w/ iOS, and I would be a little surprised if it raised issues here, but it’s probably worth having someone else test and approve this change to be safe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @samsymons!
# By Dominik Kapusta (4) and others # Via Dominik Kapusta (1) and others * main: Update Ruby to 3.3.4 (#3519) Fix UI tests failing (#3517) Add menu option to set as default browser (#3508) Bump version to 1.113.0 (300) Deprecate PixelKit daily pixel suffixes (#3509) Update permission usage description strings in Info.plist (#3518) Fix add to favorites and open in new tabs action not working on manager (#3467) Hiding of tab previews for pinned tabs fixed (#3513) Update C-S-S to 6.29.0 (#3515) Freemium PIR: Ship Review Changes - Updated New Tab Banner UI (#3501) Send pixel on sync secure storage read failure (#3497) Remove com.apple.security.device.usb entitlement (#3495) # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
# By Dominik Kapusta (4) and others # Via Dominik Kapusta (1) and others * main: VPN clean-up (#3474) Update Ruby to 3.3.4 (#3519) Fix UI tests failing (#3517) Add menu option to set as default browser (#3508) Bump version to 1.113.0 (300) Deprecate PixelKit daily pixel suffixes (#3509) Update permission usage description strings in Info.plist (#3518) Fix add to favorites and open in new tabs action not working on manager (#3467) Hiding of tab previews for pinned tabs fixed (#3513) Update C-S-S to 6.29.0 (#3515) Freemium PIR: Ship Review Changes - Updated New Tab Banner UI (#3501) Send pixel on sync secure storage read failure (#3497) Remove com.apple.security.device.usb entitlement (#3495) # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
# By Dominik Kapusta (4) and others # Via Dominik Kapusta (1) and others * main: Remove VPN test rollout pixel (#3510) VPN clean-up (#3474) Update Ruby to 3.3.4 (#3519) Fix UI tests failing (#3517) Add menu option to set as default browser (#3508) Bump version to 1.113.0 (300) Deprecate PixelKit daily pixel suffixes (#3509) Update permission usage description strings in Info.plist (#3518) Fix add to favorites and open in new tabs action not working on manager (#3467) Hiding of tab previews for pinned tabs fixed (#3513) Update C-S-S to 6.29.0 (#3515) Freemium PIR: Ship Review Changes - Updated New Tab Banner UI (#3501) Send pixel on sync secure storage read failure (#3497) Remove com.apple.security.device.usb entitlement (#3495) # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Task/Issue URL: https://app.asana.com/0/414235014887631/1208702695612235/f
Tech Design URL:
CC:
Description:
This PR updates
ruby-version
to 3.3.4, replacing the deprecated 3.0 version of Ruby that we were using previously.Steps to test this PR:
rbenv install
to ensure that 3.3.4 installs as expected (if you're already done this, you can skip this step)bundle install
to install dependenciesbundle exec fastlane lanes
to check that Fastlane worksDefinition of Done:
Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation