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 &lt;info(a)gno.de&gt; Gerrit-Reviewer: John Vandenberg &lt;jayvdb(a)gmail.com&gt; Gerrit-Reviewer: Lokal Profil &lt;andre.costa(a)wikimedia.se&gt; Gerrit-Reviewer: jenkins-bot (75)