Rails 7.1 pt2 #2316

jimbali · 2024-08-30T14:57:30Z

Jira link

MAP-1477

What?

I have added/removed/altered:

Enable most Rails 7.1 defaults
Get rid of the unnecessary guff that gets printed to the console during tests

Why?

I am doing this because:

To continue with the Rails 7.1 upgrade and get ready for 7.2
To make it easier to spot deprecation errors logged during test runs

Deployment risks (optional)

Some of these changes needed to only be deployed after all pods were running 7.1 (which they now are)

Also delete secrets.yml as it's unnecessary now and will be removed in 7.2 MAP-1477#

…le during tests This should make it easier to spot deperecation warnings. MAP-1477

Mjwillis

LGTM

jimbali added 2 commits August 30, 2024 13:41

chore: Enable most Rails 7.1 defaults

2edd94d

Also delete secrets.yml as it's unnecessary now and will be removed in 7.2 MAP-1477#

chore: Get rid of the unnecessary guff that gets printed to the conso…

852d11f

…le during tests This should make it easier to spot deperecation warnings. MAP-1477

jimbali marked this pull request as ready for review August 30, 2024 15:07

jimbali requested a review from a team as a code owner August 30, 2024 15:07

Mjwillis approved these changes Sep 2, 2024

View reviewed changes

Mjwillis merged commit a79711b into main Sep 2, 2024
8 checks passed

Mjwillis deleted the rails-7.1-pt2 branch September 2, 2024 09:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Rails 7.1 pt2 #2316

Rails 7.1 pt2 #2316

jimbali commented Aug 30, 2024

Mjwillis left a comment

Rails 7.1 pt2 #2316

Rails 7.1 pt2 #2316

Conversation

jimbali commented Aug 30, 2024

Jira link

What?

Why?

Deployment risks (optional)

Mjwillis left a comment

Choose a reason for hiding this comment