No need to :nodoc: private methods

activemodel/lib/active_model/attribute_methods.rb

@@ -349,11 +349,11 @@ def instance_method_already_implemented?(method_name) #:nodoc:
         # used to alleviate the GC, which ultimately also speeds up the app
         # significantly (in our case our test suite finishes 10% faster with
         # this cache).
-        def attribute_method_matchers_cache #:nodoc:
+        def attribute_method_matchers_cache
           @attribute_method_matchers_cache ||= Concurrent::Map.new(initial_capacity: 4)
         end
 
-        def attribute_method_matchers_matching(method_name) #:nodoc:
+        def attribute_method_matchers_matching(method_name)
           attribute_method_matchers_cache.compute_if_absent(method_name) do
             # Must try to match prefixes/suffixes first, or else the matcher with no prefix/suffix
             # will match every time.
@@ -365,7 +365,7 @@ def attribute_method_matchers_matching(method_name) #:nodoc:
         # Define a method `name` in `mod` that dispatches to `send`
         # using the given `extra` args. This falls back on `define_method`
         # and `send` if the given names cannot be compiled.
-        def define_proxy_call(include_private, mod, name, send, *extra) #:nodoc:
+        def define_proxy_call(include_private, mod, name, send, *extra)
           defn = if NAME_COMPILABLE_REGEXP.match?(name)
             "def #{name}(*args)"
           else

activemodel/lib/active_model/callbacks.rb

@@ -122,19 +122,19 @@ def define_model_callbacks(*callbacks)
 
     private
 
-      def _define_before_model_callback(klass, callback) #:nodoc:
+      def _define_before_model_callback(klass, callback)
         klass.define_singleton_method("before_#{callback}") do |*args, &block|
           set_callback(:"#{callback}", :before, *args, &block)
         end
       end
 
-      def _define_around_model_callback(klass, callback) #:nodoc:
+      def _define_around_model_callback(klass, callback)
         klass.define_singleton_method("around_#{callback}") do |*args, &block|
           set_callback(:"#{callback}", :around, *args, &block)
         end
       end
 
-      def _define_after_model_callback(klass, callback) #:nodoc:
+      def _define_after_model_callback(klass, callback)
         klass.define_singleton_method("after_#{callback}") do |*args, &block|
           options = args.extract_options!
           options[:prepend] = true