Closed kasugaijin closed 1 year ago
I think it's important to make sure that important aspects of the app are tested when upgrading.
Are there any important tests missing for the app? I personally did not find any upon a quick review.
If you are comfortable with the current test coverage, I would like to be assigned to this issue. Otherwise, I think any missing tests should be addressed first.
Yes upgrades are something to do carefully. This is a patch upgrade, so it should be a backwards compatible fix and I suspect not going to cause issues. I think the test suite is robust enough to catch any major issues.
Upgrade includes security fix for
redirect_to
which is used a lot in this application. See release notesRuby 3.1.x is still the recommended Ruby version per this table