jenkins-bot has submitted this change. ( 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/743368 )

Change subject: [IMPR] Use min buildin instead of value comparison
......................................................................

[IMPR] Use min buildin instead of value comparison

Change-Id: Ibeafad4def6af42b5374600740c93f1ca083a086
---
M scripts/checkimages.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Matěj Suchánek: Looks good to me, but someone else must approve
  Xqt: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/scripts/checkimages.py b/scripts/checkimages.py
index 08cac8a..645b25e 100755
--- a/scripts/checkimages.py
+++ b/scripts/checkimages.py
@@ -1293,8 +1293,8 @@
         if skip_number == 0:
             pywikibot.output('\t\t>> No files to skip...<<')
             return False
-        if skip_number > limit:
-            skip_number = limit
+
+        skip_number = min(skip_number, limit)
         # Print a starting message only if no images has been skipped
         if not self.skip_list:
             pywikibot.output(

--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/743368
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.wikimedia.org/r/settings

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ibeafad4def6af42b5374600740c93f1ca083a086
Gerrit-Change-Number: 743368
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: D3r1ck01 <[email protected]>
Gerrit-Reviewer: Matěj Suchánek <[email protected]>
Gerrit-Reviewer: Xqt <[email protected]>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
_______________________________________________
Pywikibot-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to