Set load_defaults to current Rails version in bug_report_templates

[zzak]

Currently only Action Mailbox and Active Storage use initialized apps, so at least in those cases we can ensure the correct defaults are used by bug reporters.

[ghiculescu]

Another question is why not use load_defaults to the latest version in the bug_report_templates?

I think this would simpler and would make the templates better reflect a real app.

[jonathanhefner]

Another question is why not use load_defaults to the latest version in the bug_report_templates?

I think this would simpler and would make the templates better reflect a real app.

Agreed. However, instead of specifying 7.1, let's use config.load_defaults Rails::VERSION::STRING.to_f, as we do when generating dummy apps. That way we don't have to update the version in the future, and the guides/bug_report_templates/*_main.rb files will pick up the 7.2 defaults.

[zzak]

Let's use load_defaults then, but actually this doesn't get rid of the cache_format_version message because the default is still set to 6.1 -- but that deprecation will be removed from main soon enough.

Still, I think part of my intention here was to reduce noise from the bug report templates which might lead contributors astray.

[jonathanhefner]

Thank you, @zzak! 👍

Let's use load_defaults then, but actually this doesn't get rid of the cache_format_version message because the default is still set to 6.1 -- but that deprecation will be removed from main soon enough.

I don't follow. config.active_support.cache_format_version is set to 7.1 when using config.load_defaults 7.1:

rails/railties/lib/rails/application/configuration.rb

Line 313 in 2745366

active_support.cache_format_version = 7.1

I ran both active_storage_gem.rb and active_storage_main.rb without this change and saw the deprecation warning. I then ran them with this change and no longer see the deprecation warning. Am I missing something?

[zzak]

Sorry, I was confused. You're correct using load_defaults will fix this.

Otherwise the default value is still set to 6.1:
https://github.com/rails/rails/blob/main/activesupport/lib/active_support/cache.rb#L55

This comes up a bit, so it's easy to forget the two. (#49543, #49892)