jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/512107 )
Change subject: [cleanup] Simplify range checking in proofreadpage.py
......................................................................
[cleanup] Simplify range checking in proofreadpage.py
Change-Id: I3cc966b44222aa2b48475b0980d6b35ceda2a2fc
---
M pywikibot/proofreadpage.py
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Lokal Profil: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/proofreadpage.py b/pywikibot/proofreadpage.py
index d05f932..4a72548 100644
--- a/pywikibot/proofreadpage.py
+++ b/pywikibot/proofreadpage.py
@@ -953,9 +953,7 @@
if end is None:
end = self.num_pages
- if not ((1 <= start <= self.num_pages)
- and (1 <= end <= self.num_pages)
- and (start <= end)):
+ if not (1 <= start <= end <= self.num_pages):
raise ValueError('start=%s, end=%s are not in valid range (%s, %s)'
% (start, end, 1, self.num_pages))
--
To view, visit
https://gerrit.wikimedia.org/r/512107
To unsubscribe, or for help writing mail filters, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3cc966b44222aa2b48475b0980d6b35ceda2a2fc
Gerrit-Change-Number: 512107
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Lokal Profil <andre.costa(a)wikimedia.se>
Gerrit-Reviewer: jenkins-bot (75)