Revert "Update Cop::Style::ModuleFunction::FORBIDDEN_MSG to suggest…

… to "Open the singleton class with `class << self` instead."." This reverts commit e6a5da52e0e57158f6902e956c5675ba8198268f.
rubocop · Oct 25, 2022 · 5f61a3b · 5f61a3b
1 parent 05a7c59
commit 5f61a3b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/changelog/change_module_function-forbidden_msg.md b/changelog/change_module_function-forbidden_msg.md
diff --git a/lib/rubocop/cop/style/module_function.rb b/lib/rubocop/cop/style/module_function.rb
@@ -76,8 +76,7 @@ class ModuleFunction < Base
 
         MODULE_FUNCTION_MSG = 'Use `module_function` instead of `extend self`.'
         EXTEND_SELF_MSG = 'Use `extend self` instead of `module_function`.'
-        FORBIDDEN_MSG = 'Do not use `module_function` or `extend self`. ' \
-                        'Open the singleton class with `class << self` instead.'
+        FORBIDDEN_MSG = 'Do not use `module_function` or `extend self`.'
 
         # @!method module_function_node?(node)
         def_node_matcher :module_function_node?, '(send nil? :module_function)'

diff --git a/spec/rubocop/cop/style/module_function_spec.rb b/spec/rubocop/cop/style/module_function_spec.rb
@@ -89,7 +89,7 @@ def test; end
         expect_offense(<<~RUBY)
           module Test
             extend self
-            ^^^^^^^^^^^ Do not use `module_function` or `extend self`. Open the singleton class with `class << self` instead.
+            ^^^^^^^^^^^ Do not use `module_function` or `extend self`.
             def test; end
           end
         RUBY
@@ -101,7 +101,7 @@ def test; end
         expect_offense(<<~RUBY)
           module Test
             extend self
-            ^^^^^^^^^^^ Do not use `module_function` or `extend self`. Open the singleton class with `class << self` instead.
+            ^^^^^^^^^^^ Do not use `module_function` or `extend self`.
             def test; end
             private
             def test_private;end
@@ -115,7 +115,7 @@ def test_private;end
         expect_offense(<<~RUBY)
           module Test
             extend self
-            ^^^^^^^^^^^ Do not use `module_function` or `extend self`. Open the singleton class with `class << self` instead.
+            ^^^^^^^^^^^ Do not use `module_function` or `extend self`.
             def test; end
             private :test
           end
@@ -137,7 +137,7 @@ class Test
       expect_offense(<<~RUBY)
         module Test
           module_function
-          ^^^^^^^^^^^^^^^ Do not use `module_function` or `extend self`. Open the singleton class with `class << self` instead.
+          ^^^^^^^^^^^^^^^ Do not use `module_function` or `extend self`.
           def test; end
         end
       RUBY