Merge pull request #284 from bastelfreak/rubocop2

rubocop: Fix Performance/StringInclude & RSpec/BeEq

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2024-05-17 10:11:09 UTC using RuboCop version 1.63.5.
+# on 2024-05-17 11:57:48 UTC using RuboCop version 1.63.5.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -12,18 +12,6 @@ Lint/SuppressedException:
   Exclude:
     - 'Rakefile'
 
-# Offense count: 1
-# This cop supports unsafe autocorrection (--autocorrect-all).
-Performance/StringInclude:
-  Exclude:
-    - 'Rakefile'
-
-# Offense count: 5
-# This cop supports unsafe autocorrection (--autocorrect-all).
-RSpec/BeEq:
-  Exclude:
-    - 'spec/unit/module_sync/settings_spec.rb'
-
 # Offense count: 3
 # Configuration parameters: Prefixes, AllowedPatterns.
 # Prefixes: when, with, without

Rakefile

@@ -37,7 +37,7 @@ begin
 
   # Workaround for https://github.com/github-changelog-generator/github-changelog-generator/issues/715
   require 'rbconfig'
-  if /linux/.match?(RbConfig::CONFIG['host_os'])
+  if RbConfig::CONFIG['host_os'].include?('linux')
     task :changelog do
       puts 'Fixing line endings...'
       changelog_file = File.join(__dir__, 'CHANGELOG.md')

spec/unit/module_sync/settings_spec.rb

@@ -13,11 +13,11 @@
     )
   end
 
-  it { is_expected.not_to eq nil }
-  it { expect(subject.managed?('Rakefile')).to eq false }
-  it { expect(subject.managed?('Rakefile/foo')).to eq false }
-  it { expect(subject.managed?('Gemfile')).to eq true }
-  it { expect(subject.managed?('Gemfile/foo')).to eq true }
+  it { is_expected.not_to be_nil }
+  it { expect(subject.managed?('Rakefile')).to be false }
+  it { expect(subject.managed?('Rakefile/foo')).to be false }
+  it { expect(subject.managed?('Gemfile')).to be true }
+  it { expect(subject.managed?('Gemfile/foo')).to be true }
   it { expect(subject.managed_files([])).to eq ['Gemfile'] }
   it { expect(subject.managed_files(%w[Rakefile Gemfile other_file])).to eq %w[Gemfile other_file] }
   it { expect(subject.unmanaged_files([])).to eq ['Rakefile'] }