fixed houndci messages

- "Redundant return detected."
- "Pass &:strip as an argument to map instead of a block."

1 files changed, 2 insertions, 2 deletions

diff --git a/app/models/job.rb b/app/models/job.rb
index b14cc33..3a358c1 100644
--- a/app/models/job.rb
+++ b/app/models/job.rb
@@ -40,7 +40,7 @@ class Job < ActiveRecord::Base
     includes = false
     # extract the refs - split by ","
     # is tricky because they can be in a regex too
-    refs.scan(/\w+|\/+.*?\/+/).map{|re| re.strip}.each do |re|
+    refs.scan(/\w+|\/+.*?\/+/).map(&:strip).each do |re|
       # is regexp or not
       if re.start_with?("/") && re.end_with?("/")
         includes = !ref.match(/#{re.delete("/")}/i).nil?
@@ -50,6 +50,6 @@ class Job < ActiveRecord::Base
 
       break if includes == true
     end
-    return includes
+    includes
   end
 end