Commit 431d7972 authored by Shinya Maeda's avatar Shinya Maeda

Fix unmatches_path

parent 1736a2da
...@@ -208,15 +208,15 @@ module Ci ...@@ -208,15 +208,15 @@ module Ci
def matching?(patterns, ref, tag, source) def matching?(patterns, ref, tag, source)
patterns.any? do |pattern| patterns.any? do |pattern|
pattern, path = pattern.split('@', 2) pattern, path = pattern.split('@', 2)
match_path?(path) && match_pattern?(pattern, ref, tag, source) unmatches_path?(path) && matches_pattern?(pattern, ref, tag, source)
end end
end end
def match_path?(path) def unmatches_path?(path)
return !(path && path != self.path) path && path != self.path
end end
def match_pattern?(pattern, ref, tag, source) def matches_pattern?(pattern, ref, tag, source)
return true if tag && pattern == 'tags' return true if tag && pattern == 'tags'
return true if !tag && pattern == 'branches' return true if !tag && pattern == 'branches'
return true if source_to_pattern(source) == pattern return true if source_to_pattern(source) == pattern
...@@ -229,8 +229,11 @@ module Ci ...@@ -229,8 +229,11 @@ module Ci
end end
def source_to_pattern(source) def source_to_pattern(source)
return source if %w(api external web).include?(source) || source.nil? if %w(api external web).include?(source) || source.nil?
return source.pluralize source
else
source.pluralize
end
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment