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

Upgrade to Rails 6.1 #203

Draft
wants to merge 3 commits into
base: staging
Choose a base branch
from
Draft

Upgrade to Rails 6.1 #203

wants to merge 3 commits into from

Conversation

patbl
Copy link
Collaborator

@patbl patbl commented Mar 10, 2021

This PR upgrades the app to 6.1.6.1, the latest released version of Rails as of this writing. According to this page, Rails 6.0.x (the version the app is currently running on) receives updates for "severe security issues" but not for "security issues." Rails 6.1.x receives updates for both "severe security issues" and "security issues." Only 7.0 is guaranteed to receive bug fixes, though I think bug fixes aren't especially important for this app.

Test plan

  • Run the specs
  • Run the app locally, click around, and make sure things seem to work. I can do a first pass, but maybe Ozzie should do more-thorough testing since he'd have a better idea about what to test.
    • Patrick does some basic testing.
    • Ozzie does some more-thorough testing.
  • Deploy this PR.
  • Deploy Fix Rails 6.1 deprecation warnings #231.

@quinn-dougherty
Copy link
Contributor

quinn-dougherty commented Jul 19, 2022

I think we get a lot of the value of security updates by going to 6.0.5 with plausibly a lot less manual testing needs #225 (epistemic status: just a devops guy knowing zilch of ruby, only remarking on the fact that it seems like you needed more substantial commits to get to 6.1 than I needed to get to 6.0.5)

@quinn-dougherty
Copy link
Contributor

This should be targeted against staging, not master.

@patbl patbl changed the base branch from master to staging September 4, 2022 17:51
We don't use it, and having it enabled makes upgrading Rails harder.
`update_attributes` will be removed in Rails 6.1. This deprecation
warning was being printed:

    update_attributes is deprecated and will be removed from Rails 6.1
    (please, use update instead)
The command that generated these changes was

    bundle update --conservative actioncable actionmailbox \
      actionmailer actionpack actiontext actionview \
      activesupport activemodel activerecord activestorage \
      rails railties tzinfo
@quinn-dougherty quinn-dougherty temporarily deployed to guesstimate-upgrade-to--orirap September 5, 2022 00:20 Inactive
@patbl
Copy link
Collaborator Author

patbl commented Sep 5, 2022

I changed the base branch, resolved the merge conflicts, and moved some non-essential commits to a separate PR.

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