-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Chore/upgrade to rails 8.1 #5162
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
base: main
Are you sure you want to change the base?
Conversation
Because: We should stay up to date on Rails versions This commit: * Run rails update task + align new configs * Set framework defaults to 8 * Update env and config files
dda1bb7 to
94a287f
Compare
|
@KevinMulhern fyi, this appears to be a relatively safe / straightforward upgrade without any (significant) breaking changes. Unblocked now that Rails 8 is in. The branch needs a bit of tidying, but I think this is a safe jump to make despite being relatively fresh. Happy to go ahead with this in the near future? Perhaps wait a week or two to see if there's any subtle issues from the last upgrade |
|
Thanks @ChargrilledChook!
💯 lets do it! Side note: is Rails 8.2 when structured logging drops? |
Nope, it's in this one 👀 I haven't played around with it much yet but keen to give it a poke |
POC / assessing effort. Blocked by #5159