Skip to content

Enable even more frozen string in app/controllers

gfyoung requested to merge (removed):frozen-string-app-controllers-much-more into master

What does this MR do?

Enable frozen string literals in:

  • app/controllers/projects/**/*.rb
  • app/controllers/sherlock/**/*.rb
  • app/controllers/snippets/**/*.rb
  • app/controllers/users/**/*.rb

Why was this MR needed?

Because frozen strings require fewer memory allocations, and memory efficiency is a good thing.

What are the relevant issue numbers?

Partially addresses #47424 (closed).

Suggested Reviewers/Approvers/Mergers

@stanhu @rymai

(@godfat is on vacation)

Merge request reports