BryanDavis has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/316515

Change subject: Remove StopIteration for PEP 479 compatibility
......................................................................

Remove StopIteration for PEP 479 compatibility

Replace 'raise StopIteration' with 'return' which is a forward and
backward compatible replacement to exit a generator.

Change-Id: I5210ec038bd98e623207001c5b8832394805db7e
---
M commit_message_validator/__init__.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/integration/commit-message-validator 
refs/changes/15/316515/1

diff --git a/commit_message_validator/__init__.py 
b/commit_message_validator/__init__.py
index bc83397..7c0f9b8 100644
--- a/commit_message_validator/__init__.py
+++ b/commit_message_validator/__init__.py
@@ -121,7 +121,7 @@
 
         if normalized_name not in FOOTERS:
             # Meh. Not a name we care about
-            raise StopIteration
+            return
 
         if normalized_name in BAD_FOOTERS:
             # Treat as the correct name for the rest of the rules

-- 
To view, visit https://gerrit.wikimedia.org/r/316515
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5210ec038bd98e623207001c5b8832394805db7e
Gerrit-PatchSet: 1
Gerrit-Project: integration/commit-message-validator
Gerrit-Branch: master
Gerrit-Owner: BryanDavis <bda...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to