Skip to content

Conversation

@radville
Copy link

@radville radville commented Sep 5, 2024

For https://github.com/github/ruby-architecture/issues/949

We are preparing to freeze all strings by default, but are getting frozen string errors from this gem. This PR marks those things as mutable so we no longer get errors.

Example error:

"/workspaces/github/vendor/gems/3.3.4/ruby/3.3.0/gems/bert-1.1.10.49.gaa7e3f5/lib/bert/decode.rb:74:in `read': can't modify frozen String: \"\" (FrozenError)\n" +

@radville radville marked this pull request as ready for review September 5, 2024 14:16
@radville radville requested a review from a team September 5, 2024 14:16
@radville radville merged commit f3b2e72 into master Sep 5, 2024
@radville radville deleted the radville/prepare-frozen-strings branch September 5, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants