Merge pull request #1445 from koic/fix_an_error_when_using_target_rails_version

koic · web-flow · commit 063bf20fef34 · 2025-02-17T16:14:54.000+09:00
[Fix #1442] Fix an incorrect behavior when using `AllCops: TargetRailsVersion`
diff --git a/changelog/fix_an_error_when_using_target_rails_version.md b/changelog/fix_an_error_when_using_target_rails_version.md
@@ -0,0 +1 @@
+* [#1442](https://github.com/rubocop/rubocop-rails/issues/1442): Fix an incorrect behavior when using `AllCops: TargetRailsVersion`. ([@koic][])
diff --git a/lib/rubocop/rails/plugin.rb b/lib/rubocop/rails/plugin.rb
@@ -24,8 +24,25 @@ def rules(_context)
 
         ConfigObsoletion.files << project_root.join('config', 'obsoletion.yml')
 
+        # FIXME: This is a dirty hack relying on a private constant to prevent
+        # "Warning: AllCops does not support TargetRailsVersion parameter".
+        # It should be updated to a better design in the future.
+        without_warnings do
+          ConfigValidator.const_set(:COMMON_PARAMS, ConfigValidator::COMMON_PARAMS.dup << 'TargetRailsVersion')
+        end
+
         LintRoller::Rules.new(type: :path, config_format: :rubocop, value: project_root.join('config', 'default.yml'))
       end
+
+      private
+
+      def without_warnings
+        original_verbose = $VERBOSE
+        $VERBOSE = nil
+        yield
+      ensure
+        $VERBOSE = original_verbose
+      end
     end
   end
 end

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+* [#1442](https://github.com/rubocop/rubocop-rails/issues/1442): Fix an incorrect behavior when using `AllCops: TargetRailsVersion`. ([@koic][])