Assume * when glob ends with forward slash

smarkets · May 5, 2020 · e27bb29 · e27bb29
1 parent 618cac6
commit e27bb29
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/marge/approvals.py b/marge/approvals.py
@@ -64,7 +64,11 @@ def determine_responsible_owners(self, owners_glob, changes):
 
         for change in changes['changes']:
             for glob, users in owners_glob.items():
-                if 'new_path' in change and fnmatch.fnmatch(change['new_path'], glob):
+                test_glob = glob
+                if glob.endswith('/'):
+                    test_glob += "*"
+
+                if 'new_path' in change and fnmatch.fnmatch(change['new_path'], test_glob):
                     owners.update(users)
 
         return owners