-
Notifications
You must be signed in to change notification settings - Fork 53
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
Upgrade ruby from 2.7.7 to 3.x latest #773
Comments
Hey @compwron, can i work on this issue? |
@fchatterji go for it! :) |
Hey, I tried to look at upgrading ruby, it creates a lot of errors in the test, that with my current knowledge of the codebase will be too long to fix. I just did the rails upgrade, will unassign myself from this issue. |
I tried looking into upgrading Ruby and found that while the app ran locally fine, all of the tests failed. I added this gem to make the tests work "gem 'net-smtp', require: false" and then there were 45 failures. I think the stripe gem needs to be upgraded first because many of the failures were around the stripe tests. |
Failures:
|
Related? #489 |
Check
.ruby-version
for what ruby we are using. Look at https://www.ruby-lang.org/en/downloads/releases/ and upgrade one version per PR until we are at the latest mainstream supported version (currently 3.2.1).The text was updated successfully, but these errors were encountered: