fix -Naming/AccessorMethodName

dtakahas · Jan 2, 2020 · c27dd57 · c27dd57
1 parent 6a7703c
commit c27dd57
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
@@ -491,11 +491,6 @@ Metrics/ModuleLength:
 Metrics/PerceivedComplexity:
   Max: 12
 
-# Offense count: 1
-Naming/AccessorMethodName:
-  Exclude:
-    - 'app/services/account_merger.rb'
-
 # Offense count: 6
 # Configuration parameters: ForbiddenDelimiters.
 # ForbiddenDelimiters: (?-mix:(^|\s)(EO[A-Z]{1}|END)(\s|$))

diff --git a/app/services/account_merger.rb b/app/services/account_merger.rb
@@ -21,7 +21,7 @@ def merge!
       Is this cool? (y/n)
     EOT
 
-    return unless get_answer.casecmp('y')
+    return unless gets.chomp.casecmp('y')
 
     to_destroy.destroy_all
 
@@ -33,10 +33,6 @@ def merge!
 
   private
 
-  def get_answer
-    gets.chomp
-  end
-
   def user_desc(user)
     "#{user.full_name} (#{user.id})"
   end