Load rails 7.0 defaults

[segiddins]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.15%. Comparing base (be15c54) to head (ffc0674).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4388      +/-   ##
==========================================
- Coverage   97.15%   97.15%   -0.01%     
==========================================
  Files         391      391              
  Lines        8258     8257       -1     
==========================================
- Hits         8023     8022       -1     
  Misses        235      235              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[segiddins]

0 clue why this would cause test failures, it should be no functional change

[martinemde]

That's strange. Could it be that other gems are relying on the old api? For example, what's with the deprecation all the sudden?

DEPRECATION WARNING: `Rails.application.secrets` is deprecated in favor of `Rails.application.credentials` and will be removed in Rails 7.2. (called from call at /home/runner/work/rubygems.org/rubygems.org/lib/clearance_backdoor.rb:9)

Either I'm crazy or that code doesn't have a reference to secrets (when I look again maybe it is the call to clearance).

[segiddins]

I believe the warnings are unrelated

[simi]

@segiddins @martinemde something in this list https://github.com/rails/rails/blob/4d3e2910c680baad7c14d2797a53a6b72405149b/railties/lib/rails/application/configuration.rb#L219-L268 causes find_by_sql calls to be cached and sadly there is code calling UPDATE clauses with find_by_sql which are not executed sometimes due to this change.

There were 2 places using find_by_sql, I have refactored them to use update_all or update_column. See each individual commit message for more details.