Ruby 4 intends to make string literals frozen by default and to aid transition it is possible to opt into that behaviour on a per-file basis by adding a special comment to the top, as this PR does.
Note that ruby 3.4 already warns if you try and modify a non-frozen string literal and you have deprecation warnings enabled (which our testing framework does by default) so this will help us avoid introducing such warnings by effectively making them a hard error instead. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5932 -- Commit Summary -- * Add frozen_string_literal comments to ruby files -- File Changes -- M .rubocop.yml (4) M .rubocop_todo.yml (7) M Dangerfile (2) M Gemfile (2) M Rakefile (2) M Vagrantfile (2) M app/controllers/accounts/deletions_controller.rb (2) M app/controllers/accounts/homes_controller.rb (2) M app/controllers/accounts/pd_declarations_controller.rb (2) M app/controllers/accounts/terms_controller.rb (2) M app/controllers/accounts_controller.rb (2) M app/controllers/api/capabilities_controller.rb (2) M app/controllers/api/changeset_comments/visibilities_controller.rb (2) M app/controllers/api/changeset_comments_controller.rb (2) M app/controllers/api/changeset_subscriptions_controller.rb (2) M app/controllers/api/changesets/downloads_controller.rb (2) M app/controllers/api/changesets_controller.rb (2) M app/controllers/api/maps_controller.rb (2) M app/controllers/api/messages/inboxes_controller.rb (2) M app/controllers/api/messages/mailboxes_controller.rb (2) M app/controllers/api/messages/outboxes_controller.rb (2) M app/controllers/api/messages_controller.rb (2) M app/controllers/api/nodes/relations_controller.rb (2) M app/controllers/api/nodes/ways_controller.rb (2) M app/controllers/api/nodes_controller.rb (2) M app/controllers/api/note_subscriptions_controller.rb (2) M app/controllers/api/notes_controller.rb (2) M app/controllers/api/old_elements/redactions_controller.rb (2) M app/controllers/api/old_elements_controller.rb (2) M app/controllers/api/old_nodes/redactions_controller.rb (2) M app/controllers/api/old_nodes_controller.rb (2) M app/controllers/api/old_relations/redactions_controller.rb (2) M app/controllers/api/old_relations_controller.rb (2) M app/controllers/api/old_ways/redactions_controller.rb (2) M app/controllers/api/old_ways_controller.rb (2) M app/controllers/api/permissions_controller.rb (2) M app/controllers/api/relations/relations_controller.rb (2) M app/controllers/api/relations_controller.rb (2) M app/controllers/api/tracepoints_controller.rb (2) M app/controllers/api/traces/data_controller.rb (2) M app/controllers/api/traces_controller.rb (2) M app/controllers/api/user_blocks/active_lists_controller.rb (2) M app/controllers/api/user_blocks_controller.rb (2) M app/controllers/api/user_preferences_controller.rb (2) M app/controllers/api/users/traces_controller.rb (2) M app/controllers/api/users_controller.rb (2) M app/controllers/api/versions_controller.rb (2) M app/controllers/api/ways/relations_controller.rb (2) M app/controllers/api/ways_controller.rb (2) M app/controllers/api_controller.rb (2) M app/controllers/application_controller.rb (2) M app/controllers/changeset_comments/feeds_controller.rb (2) M app/controllers/changeset_subscriptions_controller.rb (2) M app/controllers/changesets_controller.rb (2) M app/controllers/concerns/email_methods.rb (2) M app/controllers/concerns/pagination_methods.rb (2) M app/controllers/concerns/query_methods.rb (2) M app/controllers/concerns/session_methods.rb (2) M app/controllers/concerns/user_methods.rb (2) M app/controllers/confirmations_controller.rb (2) M app/controllers/dashboards_controller.rb (2) M app/controllers/diary_comments_controller.rb (2) M app/controllers/diary_entries_controller.rb (2) M app/controllers/directions_controller.rb (2) M app/controllers/elements_controller.rb (2) M app/controllers/errors_controller.rb (2) M app/controllers/export_controller.rb (2) M app/controllers/feature_queries_controller.rb (2) M app/controllers/follows_controller.rb (2) M app/controllers/geocoder_controller.rb (2) M app/controllers/issue_comments_controller.rb (2) M app/controllers/issues/reporters_controller.rb (2) M app/controllers/issues_controller.rb (2) M app/controllers/map_keys_controller.rb (2) M app/controllers/messages/inboxes_controller.rb (2) M app/controllers/messages/mailboxes_controller.rb (2) M app/controllers/messages/muted_inboxes_controller.rb (2) M app/controllers/messages/mutes_controller.rb (2) M app/controllers/messages/outboxes_controller.rb (2) M app/controllers/messages/read_marks_controller.rb (2) M app/controllers/messages/replies_controller.rb (2) M app/controllers/messages_controller.rb (2) M app/controllers/nodes_controller.rb (2) M app/controllers/notes_controller.rb (2) M app/controllers/oauth2_applications_controller.rb (2) M app/controllers/oauth2_authorizations_controller.rb (2) M app/controllers/oauth2_authorized_applications_controller.rb (2) M app/controllers/old_elements_controller.rb (2) M app/controllers/old_nodes_controller.rb (2) M app/controllers/old_relations_controller.rb (2) M app/controllers/old_ways_controller.rb (2) M app/controllers/passwords_controller.rb (2) M app/controllers/preferences_controller.rb (2) M app/controllers/profiles_controller.rb (2) M app/controllers/redactions_controller.rb (2) M app/controllers/relations_controller.rb (2) M app/controllers/reports_controller.rb (2) M app/controllers/sessions_controller.rb (2) M app/controllers/site_controller.rb (2) M app/controllers/traces/data_controller.rb (2) M app/controllers/traces/feeds_controller.rb (2) M app/controllers/traces/icons_controller.rb (2) M app/controllers/traces/pictures_controller.rb (2) M app/controllers/traces_controller.rb (2) M app/controllers/user_blocks_controller.rb (2) M app/controllers/user_mutes_controller.rb (2) M app/controllers/user_roles_controller.rb (2) M app/controllers/users/changeset_comments_controller.rb (2) M app/controllers/users/comments_controller.rb (2) M app/controllers/users/diary_comments_controller.rb (2) M app/controllers/users/issued_blocks_controller.rb (2) M app/controllers/users/lists_controller.rb (2) M app/controllers/users/received_blocks_controller.rb (2) M app/controllers/users/statuses_controller.rb (2) M app/controllers/users_controller.rb (2) M app/controllers/ways_controller.rb (2) M app/helpers/application_helper.rb (2) M app/helpers/asset_helper.rb (2) M app/helpers/authorization_helper.rb (2) M app/helpers/banner_helper.rb (2) M app/helpers/browse_helper.rb (2) M app/helpers/browse_tags_helper.rb (4) M app/helpers/changesets_helper.rb (2) M app/helpers/geocoder_helper.rb (2) M app/helpers/issues_helper.rb (2) M app/helpers/note_helper.rb (2) M app/helpers/open_graph_helper.rb (2) M app/helpers/reports_helper.rb (2) M app/helpers/social_share_button_helper.rb (2) M app/helpers/svg_helper.rb (2) M app/helpers/title_helper.rb (2) M app/helpers/trace_helper.rb (2) M app/helpers/user_blocks_helper.rb (2) M app/helpers/user_helper.rb (2) M app/helpers/user_mailer_helper.rb (2) M app/helpers/user_roles_helper.rb (2) M app/jobs/application_job.rb (2) M app/jobs/trace_destroyer_job.rb (2) M app/jobs/trace_importer_job.rb (2) M app/mailers/application_mailer.rb (2) M app/mailers/user_mailer.rb (2) M app/models/acl.rb (2) M app/models/application_record.rb (2) M app/models/changeset.rb (2) M app/models/changeset_comment.rb (2) M app/models/changeset_subscription.rb (2) M app/models/changeset_tag.rb (2) M app/models/community.rb (2) M app/models/concerns/consistency_validations.rb (2) M app/models/concerns/geo_record.rb (2) M app/models/concerns/not_redactable.rb (2) M app/models/concerns/redactable.rb (2) M app/models/diary_comment.rb (2) M app/models/diary_entry.rb (2) M app/models/diary_entry_subscription.rb (2) M app/models/follow.rb (2) M app/models/issue.rb (2) M app/models/issue_comment.rb (2) M app/models/language.rb (2) M app/models/message.rb (2) M app/models/node.rb (2) M app/models/node_tag.rb (2) M app/models/note.rb (2) M app/models/note_comment.rb (2) M app/models/note_subscription.rb (2) M app/models/oauth2_application.rb (2) M app/models/old_node.rb (2) M app/models/old_node_tag.rb (2) M app/models/old_relation.rb (2) M app/models/old_relation_member.rb (2) M app/models/old_relation_tag.rb (2) M app/models/old_way.rb (2) M app/models/old_way_node.rb (2) M app/models/old_way_tag.rb (2) M app/models/redaction.rb (2) M app/models/relation.rb (2) M app/models/relation_member.rb (2) M app/models/relation_tag.rb (2) M app/models/report.rb (2) M app/models/trace.rb (2) M app/models/tracepoint.rb (2) M app/models/tracetag.rb (2) M app/models/user.rb (2) M app/models/user_block.rb (2) M app/models/user_mute.rb (2) M app/models/user_preference.rb (2) M app/models/user_role.rb (2) M app/models/way.rb (2) M app/models/way_node.rb (2) M app/models/way_tag.rb (2) M app/validators/characters_validator.rb (6) M app/validators/image_validator.rb (2) M app/validators/normalized_uniqueness_validator.rb (2) M app/validators/utf8_validator.rb (2) M app/validators/whitespace_validator.rb (2) M app/validators/width_validator.rb (2) M app/views/api/_root_attributes.json.jbuilder (2) M app/views/api/capabilities/show.builder (2) M app/views/api/capabilities/show.json.jbuilder (2) M app/views/api/changeset_comments/_changeset_comment.json.jbuilder (2) M app/views/api/changeset_comments/_changeset_comment.xml.builder (2) M app/views/api/changeset_comments/index.json.jbuilder (2) M app/views/api/changeset_comments/index.xml.builder (2) M app/views/api/changeset_comments/visibilities/create.json.jbuilder (2) M app/views/api/changeset_comments/visibilities/create.xml.builder (2) M app/views/api/changeset_comments/visibilities/destroy.json.jbuilder (2) M app/views/api/changeset_comments/visibilities/destroy.xml.builder (2) M app/views/api/changeset_subscriptions/create.json.jbuilder (2) M app/views/api/changeset_subscriptions/create.xml.builder (2) M app/views/api/changeset_subscriptions/destroy.json.jbuilder (2) M app/views/api/changeset_subscriptions/destroy.xml.builder (2) M app/views/api/changesets/_changeset.json.jbuilder (2) M app/views/api/changesets/_changeset.xml.builder (2) M app/views/api/changesets/downloads/show.xml.builder (2) M app/views/api/changesets/index.json.jbuilder (2) M app/views/api/changesets/index.xml.builder (2) M app/views/api/changesets/show.json.jbuilder (2) M app/views/api/changesets/show.xml.builder (2) M app/views/api/maps/_bounds.json.jbuilder (2) M app/views/api/maps/_bounds.xml.builder (2) M app/views/api/maps/show.json.jbuilder (2) M app/views/api/maps/show.xml.builder (2) M app/views/api/messages/_message.json.jbuilder (2) M app/views/api/messages/_message.xml.builder (2) M app/views/api/messages/mailboxes/show.json.jbuilder (2) M app/views/api/messages/mailboxes/show.xml.builder (2) M app/views/api/messages/show.json.jbuilder (2) M app/views/api/messages/show.xml.builder (2) M app/views/api/nodes/_node.json.jbuilder (2) M app/views/api/nodes/_node.xml.builder (2) M app/views/api/nodes/index.json.jbuilder (2) M app/views/api/nodes/index.xml.builder (2) M app/views/api/nodes/relations/index.json.jbuilder (2) M app/views/api/nodes/relations/index.xml.builder (2) M app/views/api/nodes/show.json.jbuilder (2) M app/views/api/nodes/show.xml.builder (2) M app/views/api/nodes/ways/index.json.jbuilder (2) M app/views/api/nodes/ways/index.xml.builder (2) M app/views/api/notes/_note.gpx.builder (2) M app/views/api/notes/_note.json.jbuilder (2) M app/views/api/notes/_note.rss.builder (2) M app/views/api/notes/_note.xml.builder (2) M app/views/api/notes/feed.rss.builder (2) M app/views/api/notes/index.gpx.builder (2) M app/views/api/notes/index.json.jbuilder (2) M app/views/api/notes/index.rss.builder (2) M app/views/api/notes/index.xml.builder (2) M app/views/api/notes/show.gpx.builder (2) M app/views/api/notes/show.json.jbuilder (2) M app/views/api/notes/show.rss.builder (2) M app/views/api/notes/show.xml.builder (2) M app/views/api/old_nodes/_old_node.json.jbuilder (2) M app/views/api/old_nodes/_old_node.xml.builder (2) M app/views/api/old_nodes/index.json.jbuilder (2) M app/views/api/old_nodes/index.xml.builder (2) M app/views/api/old_nodes/show.json.jbuilder (2) M app/views/api/old_nodes/show.xml.builder (2) M app/views/api/old_relations/_old_relation.json.jbuilder (2) M app/views/api/old_relations/_old_relation.xml.builder (2) M app/views/api/old_relations/index.json.jbuilder (2) M app/views/api/old_relations/index.xml.builder (2) M app/views/api/old_relations/show.json.jbuilder (2) M app/views/api/old_relations/show.xml.builder (2) M app/views/api/old_ways/_old_way.json.jbuilder (2) M app/views/api/old_ways/_old_way.xml.builder (2) M app/views/api/old_ways/index.json.jbuilder (2) M app/views/api/old_ways/index.xml.builder (2) M app/views/api/old_ways/show.json.jbuilder (2) M app/views/api/old_ways/show.xml.builder (2) M app/views/api/permissions/show.json.jbuilder (2) M app/views/api/permissions/show.xml.builder (2) M app/views/api/relations/_relation.json.jbuilder (2) M app/views/api/relations/_relation.xml.builder (2) M app/views/api/relations/index.json.jbuilder (2) M app/views/api/relations/index.xml.builder (2) M app/views/api/relations/relations/index.json.jbuilder (2) M app/views/api/relations/relations/index.xml.builder (2) M app/views/api/relations/show.json.jbuilder (2) M app/views/api/relations/show.xml.builder (2) M app/views/api/tracepoints/index.gpx.builder (2) M app/views/api/traces/_trace.builder (2) M app/views/api/traces/show.builder (2) M app/views/api/user_blocks/_user_block.json.jbuilder (2) M app/views/api/user_blocks/_user_block.xml.builder (2) M app/views/api/user_blocks/active_lists/show.json.jbuilder (2) M app/views/api/user_blocks/active_lists/show.xml.builder (2) M app/views/api/user_blocks/show.json.jbuilder (2) M app/views/api/user_blocks/show.xml.builder (2) M app/views/api/user_preferences/_user_preference.xml.builder (2) M app/views/api/user_preferences/index.json.jbuilder (2) M app/views/api/user_preferences/index.xml.builder (2) M app/views/api/users/_user.json.jbuilder (2) M app/views/api/users/_user.xml.builder (2) M app/views/api/users/index.json.jbuilder (2) M app/views/api/users/index.xml.builder (2) M app/views/api/users/show.json.jbuilder (2) M app/views/api/users/show.xml.builder (2) M app/views/api/users/traces/index.builder (2) M app/views/api/versions/show.builder (2) M app/views/api/versions/show.json.jbuilder (2) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/5932.patch https://github.com/openstreetmap/openstreetmap-website/pull/5932.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5932 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________ rails-dev mailing list rails-dev@openstreetmap.org https://lists.openstreetmap.org/listinfo/rails-dev