Skip to content

feat: Upgrade to Rails 7.2#259

Merged
rzane merged 4 commits intoBetterment:mainfrom
Irving-Betterment:icaro/rails_7_2_upgrade
Aug 6, 2025
Merged

feat: Upgrade to Rails 7.2#259
rzane merged 4 commits intoBetterment:mainfrom
Irving-Betterment:icaro/rails_7_2_upgrade

Conversation

@Irving-Betterment
Copy link
Contributor

@Irving-Betterment Irving-Betterment commented Jul 29, 2025

Summary

This pull request bumps the application to Rails 7.2; it also updates its dependencies, addressing Rails 8.0 related deprecation warnings.

Other Information

I introduced a well-known patch to appease devise reporting Rails related deprecation warnings, until the fix is released by the gem.

/domain @Betterment/test_track_core

@Irving-Betterment Irving-Betterment force-pushed the icaro/rails_7_2_upgrade branch from 6db1f65 to 5ffb90a Compare July 31, 2025 15:38
@Irving-Betterment Irving-Betterment force-pushed the icaro/rails_7_2_upgrade branch from 5ffb90a to e5d07d6 Compare July 31, 2025 15:40
Fixing `DEPRECATION WARNING: ActiveSupport::ProxyObject is deprecated and will be removed in Rails 8.0.`

# Avoid deprecation notices by pinning mail to Ruby 2.7.5 compatible version
gem 'mail', '~> 2.7.1'
gem 'mail'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unpinning this gem now that it does not longer support Ruby 2.7.5, allowing to bump up to a version supporting Rails 7.2+.

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 app doesn't call secrets explicitly itself; and I am promoting a monkey patch to appease the deprecation coming from devise.

@Irving-Betterment Irving-Betterment marked this pull request as ready for review July 31, 2025 16:46
Fix "DEPRECATION WARNING: `rails/console/helpers` has been deprecated and will be removed in Rails 8.0."
Copy link
Contributor

@rzane rzane left a comment

Choose a reason for hiding this comment

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

domain lgtm
platform lgtm

@rzane rzane merged commit e0129e3 into Betterment:main Aug 6, 2025
4 checks passed
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.

2 participants