[openstreetmap/openstreetmap-website] Add frozen_string_literal comments to ruby files (PR #5932)

Nicholas La Roux notifications at github.com
Mon Aug 18 14:45:27 UTC 2025


larouxn left a comment (openstreetmap/openstreetmap-website#5932)

I second this change. It's been best practice to add these `frozen_string_literal` comments for a little while now as a way of preparing for Ruby's eventual enforced string freezing. Even supposing we can remove the comments in Ruby 4, at least we'll know the codebase is ready. Plus, we'll probably able to auto-remove them via RuboCop ( same as how we're adding them here, nice :clap: ) or a clever global search and replace since they're always the first line followed by a newline.

:wave: @tomhughes, feel free to let me know if there's anything I can do to help push this forward. :rocket: 

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

Message ID: <openstreetmap/openstreetmap-website/pull/5932/c3197236841 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250818/e48a258d/attachment-0001.htm>


More information about the rails-dev mailing list