Skip to content

Enable frozen string in app/controllers/**/*.rb

gfyoung requested to merge (removed):frozen-string-app-controller into master

What does this MR do?

Enable frozen string literals in:

  • app/controllers/*.rb
  • app/controllers/admin/**/*.rb
  • app/controllers/boards/**/*.rb
  • app/controllers/ci/**/*.rb
  • app/controllers/concerns/**/*.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 @godfat @rymai

Merge request reports