Follow-up from "Add usage_metric_definition generator to generate new yaml metric file"
The following discussion from !51201 (merged) should be addressed:
-
@alexkalderimis started a discussion: minor:
I just realized that we ought to freeze these structures, as they are intended to be immutable.
def initialize(*args) super freeze end def match?(str)
we never mutate these objects, but since we store them in a constant, they themselves need to be frozen too.