Add frozen_string_literal to files

dentarg · dentarg · commit 7c260c614807 · 2025-08-02T18:09:25.000+02:00
Removes lots of warnings when running the tests.
diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source "https://www.rubygems.org"
  
 gemspec
diff --git a/Rakefile b/Rakefile
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake/testtask'
 require 'rdoc/task'
 
diff --git a/lib/rubygems/commands/compare_command.rb b/lib/rubygems/commands/compare_command.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems/command'
 require 'rubygems/version_option'
 require 'rubygems/comparator'
diff --git a/lib/rubygems/comparator.rb b/lib/rubygems/comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'tmpdir'
 require 'rbconfig'
 require 'rainbow'
diff --git a/lib/rubygems/comparator/base.rb b/lib/rubygems/comparator/base.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rainbow'
 require 'rubygems/comparator/utils'
 require 'rubygems/comparator/report'
diff --git a/lib/rubygems/comparator/dependency_comparator.rb b/lib/rubygems/comparator/dependency_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems/comparator/base'
 
 class Gem::Comparator
diff --git a/lib/rubygems/comparator/dir_utils.rb b/lib/rubygems/comparator/dir_utils.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'pathname'
 
 class Gem::Comparator
diff --git a/lib/rubygems/comparator/file_list_comparator.rb b/lib/rubygems/comparator/file_list_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'diffy'
 require 'rubygems/comparator/base'
 require 'rubygems/comparator/dir_utils'
diff --git a/lib/rubygems/comparator/gemfile_comparator.rb b/lib/rubygems/comparator/gemfile_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gemnasium/parser'
 require 'rubygems/comparator/base'
 
diff --git a/lib/rubygems/comparator/monitor.rb b/lib/rubygems/comparator/monitor.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'diffy'
 require 'rubygems/comparator/base'
 require 'rubygems/comparator/dir_utils'
@@ -15,7 +17,7 @@ def self.lines_changed(prev_file, curr_file)
 
     def self.compact_files_diff(prev_file, curr_file)
       prev_file = prev_file.nil? ? Tempfile.new.path : prev_file
-      changes = ''
+      changes = +''
       Diffy::Diff.new(
         prev_file, curr_file, :source => 'files', :context => 0
       ).each do |line|
@@ -29,7 +31,7 @@ def self.compact_files_diff(prev_file, curr_file)
 
     def self.files_diff(prev_file, curr_file)
       prev_file = prev_file.nil? ? Tempfile.new.path : prev_file
-      changes = ''
+      changes = +''
       Diffy::Diff.new(
         prev_file, curr_file, :source => 'files', :context => 0, :include_diff_info => true
       ).each do |line|
diff --git a/lib/rubygems/comparator/report.rb b/lib/rubygems/comparator/report.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems/comparator/report/entry'
 
 class Gem::Comparator
diff --git a/lib/rubygems/comparator/report/entry.rb b/lib/rubygems/comparator/report/entry.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Gem::Comparator
   class Report
     class Entry
diff --git a/lib/rubygems/comparator/spec_comparator.rb b/lib/rubygems/comparator/spec_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems/comparator/base'
 
 class Gem::Comparator
diff --git a/lib/rubygems/comparator/utils.rb b/lib/rubygems/comparator/utils.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rainbow'
 require 'rubygems'
 require 'rubygems/user_interaction'
diff --git a/lib/rubygems/comparator/version.rb b/lib/rubygems/comparator/version.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Gem::Comparator
   VERSION = '1.2.1'
 end
diff --git a/lib/rubygems_plugin.rb b/lib/rubygems_plugin.rb
@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require 'rubygems/command_manager'
 Gem::CommandManager.instance.register_command :compare
diff --git a/test/rubygems/comparator/test_dependency_comparator.rb b/test/rubygems/comparator/test_dependency_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestDependencyComparator < TestGemComparator
diff --git a/test/rubygems/comparator/test_dir_utils.rb b/test/rubygems/comparator/test_dir_utils.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestDirUtils < TestGemModule
diff --git a/test/rubygems/comparator/test_file_list_comparator.rb b/test/rubygems/comparator/test_file_list_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestFileListComparator < TestGemComparator
diff --git a/test/rubygems/comparator/test_gemfile_comparator.rb b/test/rubygems/comparator/test_gemfile_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestGemfileComparator < TestGemComparator
diff --git a/test/rubygems/comparator/test_monitor.rb b/test/rubygems/comparator/test_monitor.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestMonitor < TestGemModule
diff --git a/test/rubygems/comparator/test_report.rb b/test/rubygems/comparator/test_report.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../../test_helper'
 require 'rubygems/comparator'
 
diff --git a/test/rubygems/comparator/test_spec_comparator.rb b/test/rubygems/comparator/test_spec_comparator.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestSpecComparator < TestGemComparator
diff --git a/test/rubygems/comparator/test_utils.rb b/test/rubygems/comparator/test_utils.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../../test_helper'
 require 'rubygems/comparator'
 
diff --git a/test/rubygems/test_gem_commands_compare_command.rb b/test/rubygems/test_gem_commands_compare_command.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../test_helper'
 require_relative 'mock_gem_ui'
 require 'rubygems/user_interaction'
diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'minitest/autorun'
 require 'rubygems/comparator'
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`source "https://www.rubygems.org"`
`2`	`4`
`3`	`5`	`gemspec`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'rake/testtask'`
`2`	`4`	`require 'rdoc/task'`
`3`	`5`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'rubygems/command'`
`2`	`4`	`require 'rubygems/version_option'`
`3`	`5`	`require 'rubygems/comparator'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'tmpdir'`
`2`	`4`	`require 'rbconfig'`
`3`	`5`	`require 'rainbow'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'rubygems/comparator/base'`
`2`	`4`
`3`	`5`	`class Gem::Comparator`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'pathname'`
`2`	`4`
`3`	`5`	`class Gem::Comparator`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'diffy'`
`2`	`4`	`require 'rubygems/comparator/base'`
`3`	`5`	`require 'rubygems/comparator/dir_utils'`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+# frozen_string_literal: true`
	`2`	`+`
`1`	`3`	`require 'gemnasium/parser'`
`2`	`4`	`require 'rubygems/comparator/base'`
`3`	`5`