Skip to content

Enable frozen string for app/workers/**/*.rb

gfyoung requested to merge (removed):frozen-string-enable-app-workers-2 into master

What does this MR do?

Enable frozen string literals ALL Ruby files in app/workers.

This is a smaller MR compared to !19944 (merged), but for atomicity purposes, I wanted to complete one directory at a time.

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