Selectively disabled cops for methods and classes

Rubocop now requires that cops are scoped tighter. This reformats it so that only the specific method or class has the cop disabled.
httprb · May 7, 2019 · 4ceb3ee · 4ceb3ee
1 parent fb6d66e
commit 4ceb3ee
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 8 deletions.
diff --git a/lib/http/options.rb b/lib/http/options.rb
@@ -1,14 +1,12 @@
 # frozen_string_literal: true
 
-# rubocop:disable Metrics/ClassLength
-
 require "http/headers"
 require "openssl"
 require "socket"
 require "http/uri"
 
 module HTTP
-  class Options
+  class Options # rubocop:disable Metrics/ClassLength
     @default_socket_class     = TCPSocket
     @default_ssl_socket_class = OpenSSL::SSL::SSLSocket
     @default_timeout_class    = HTTP::Timeout::Null

diff --git a/spec/support/black_hole.rb b/spec/support/black_hole.rb
@@ -2,11 +2,9 @@
 
 module BlackHole
   class << self
-    # rubocop:disable Style/MethodMissingSuper
-    def method_missing(*)
+    def method_missing(*) # rubocop:disable Style/MethodMissingSuper
       self
     end
-    # rubocop:enable Style/MethodMissingSuper
 
     def respond_to_missing?(*)
       true

diff --git a/spec/support/dummy_server/servlet.rb b/spec/support/dummy_server/servlet.rb
@@ -2,8 +2,7 @@
 # frozen_string_literal: true
 
 class DummyServer < WEBrick::HTTPServer
-  # rubocop:disable Metrics/ClassLength
-  class Servlet < WEBrick::HTTPServlet::AbstractServlet
+  class Servlet < WEBrick::HTTPServlet::AbstractServlet # rubocop:disable Metrics/ClassLength
     def self.sockets
       @sockets ||= []
     end