jenkins-bot submitted this change.

View Change

Approvals: jenkins-bot: Verified Xqt: Looks good to me, approved
[redirect] raise error explicitly in _skip_on_exception

explicit is better than implicit here;
this also solves some linter errors

Change-Id: Ic5055982744ddcacea2d07c13e97cef377f5f1b9
---
M scripts/redirect.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/redirect.py b/scripts/redirect.py
index 61a0d6c..c415c2d 100755
--- a/scripts/redirect.py
+++ b/scripts/redirect.py
@@ -599,7 +599,7 @@
# all uncatched exceptions
# Note: elif statements are necessary above because all Errors
# above derive from Exception class
- raise
+ raise error

return True


To view, visit change 1052703. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ic5055982744ddcacea2d07c13e97cef377f5f1b9
Gerrit-Change-Number: 1052703
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: D3r1ck01 <dalangi-ctr@wikimedia.org>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot