Merge pull request #508 from casperisfine/enable-frozen-string-literal

Ensure compatibility with `ruby --enable-frozen-string-literal`
voxpupuli · Mar 15, 2024 · 7e4c49a · 7e4c49a
2 parents 2cc181b + d1d1e48
commit 7e4c49a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -27,6 +27,9 @@ jobs:
           - ruby: "3.1"
             coverage: "yes"
           - ruby: "3.2"
+          - ruby: "3.3"
+          - ruby: "3.3"
+            rubyopt: "--enable-frozen-string-literal"
     env:
       COVERAGE: ${{ matrix.coverage }}
     steps:
@@ -37,7 +40,7 @@ jobs:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
       - name: Run tests
-        run: bundle exec rake ${{ matrix.rake_task }}
+        run: bundle exec rake ${{ matrix.rake_task }} RUBYOPT="${{ matrix.rubyopt }}"
       - name: Build gem
         run: gem build *.gemspec
   tests:

diff --git a/lib/json-schema/util/uuid.rb b/lib/json-schema/util/uuid.rb
@@ -46,7 +46,7 @@ def mask19 v, str
           # nstr[7] |= 0b01010000
           nstr[8] &= 0b00111111
           nstr[8] |= 0b10000000
-          str = ''
+          str = +''
           nstr.each { |s| str << s.chr }
           str
         end