Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fast on the heels of 7.0 here comes the latest and greatest Rails version! # What it does Gets us on Rails 7.1. # Why it is important Latest is greatest. And specifically this gets us access to background preprocessing of variants for finishing #1150 # Implementation notes * This gets us off the fork of `audited` we were using. The [change we switched for was upstreamed](collectiveidea/audited#609) and we need the [more recent work to make it compatible with Rails 7.1](collectiveidea/audited#686). Just a couple of code changes were needed and tests seem to be passing okay. * Updates us to [Devise 4.9.3](https://github.com/heartcombo/devise/blob/main/CHANGELOG.md#493---2023-10-11) which added Rails 7.1 support. - Note Devise 4.9.0 [added Hotwire + Turbo support](https://github.com/heartcombo/devise/blob/main/CHANGELOG.md#490---2023-02-17), something we might want to take a look at in either #1184 or a follow on task. * Includes a bump to Postgres 15 for the Docker and CI setups to match production. * The new Rails defaults in 7.1 raise in controllers when you mention an action that is not defined. This caught a couple of `before_action` typos. * Running `rake app:update` yielded diffs on many of the config files. I took all the changes to `bin` scripts whole hog and merged the rest. A second set of eyes across those diffs would be helpful though! 👀 * 💥 The upgrade caused an issue with threaded background work in tests whose fix is at the bottom of `config/environments/test.rb` with a note explaining. * 💡 I had neglected to update the Yarn dependencies for Rails in the 7 upgrade, so catching those up here. * 🚧 Asset compilation now runs on every execution of `bin/rails test` which made the SCSS deprecation warnings get pretty annoying. Pulling in a workaround from picturepan2/spectre#694 * Zeitwork (the new code autoloader) did not like `lib/rails_solargraph.rb`, so following a [comment on the referenced gist](https://gist.github.com/castwide/28b349566a223dfb439a337aea29713e?permalink_comment_id=4346127#gistcomment-4346127) I switch us to the solargraph-rails gem.
- Loading branch information