Skip to content

EE Port of gitlab-ce!21736 (Enable frozen string in app/controllers/**/*.rb)

gfyoung requested to merge (removed):ee-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 gitlab-ce#47424.

Port of gitlab-ce!21320

Suggested Reviewers/Approvers/Mergers

@stanhu @godfat @rymai

Edited by Stan Hu

Merge request reports