lib/fourchette/pull_request.rb in fourchette-0.1.2 vs lib/fourchette/pull_request.rb in fourchette-0.1.3

@@ -1,9 +1,9 @@
 class Fourchette::PullRequest
   include SuckerPunch::Job
 
-  def perform params
+  def perform(params)
     return if qa_skip?(params)
 
     callbacks = Fourchette::Callbacks.new(params)
     fork = Fourchette::Fork.new(params)
 
@@ -23,10 +23,10 @@
     callbacks.after_all
   end
 
   private
 
-  def qa_skip? params
-    params['pull_request']['title'].downcase.include?('[qa skip]')
+  def qa_skip?(params)
+    pr_title = params['pull_request']['title']
+    pr_title.downcase.include?('[qa skip]')
   end
-
 end