[openstreetmap/openstreetmap-website] Switch to rails 7.2 defaults (PR #5247)

Tom Hughes notifications at github.com
Wed Oct 2 18:21:57 UTC 2024


There doesn't seem to be anything in the new defaults that is likely to cause us a problem.

* `enqueue_after_transaction_commit` probably gets disabled by delayed_job in production anyway
* `web_image_content_types` our production libvips has webp support
* `validate_migration_timestamps` shouldn't affect us as we don't have future dated migrations
* `postgresql_adapter_decode_dates` shouldn't affect us as we don't use raw queries like that
* `yjit` doesn't affect production as we're not using 3.3 but doesn't seem to cause any problems in my 3.3 environment
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5247

-- Commit Summary --

  * Switch to rails 7.2 defaults

-- File Changes --

    M config/application.rb (2)
    D config/initializers/new_framework_defaults_7_2.rb (70)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5247.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5247.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5247
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5247 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20241002/8a1085db/attachment-0001.htm>


More information about the rails-dev mailing list