jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/507352 )
Change subject: [IMPR] Improvements for arg parsing in commonscat.py
......................................................................
[IMPR] Improvements for arg parsing in commonscat.py
Step 2 of commonscat.py improvements
Change-Id: I343e59706266968894e854c99e7783c6e0a321cc
---
M scripts/commonscat.py
1 file changed, 8 insertions(+), 9 deletions(-)
Approvals:
Framawiki: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/commonscat.py b/scripts/commonscat.py
index 1be4a49..889c5f8 100755
--- a/scripts/commonscat.py
+++ b/scripts/commonscat.py
@@ -526,16 +526,15 @@
genFactory = pagegenerators.GeneratorFactory()
for arg in local_args:
- if arg.startswith('-summary'):
- if len(arg) == 8:
- options['summary'] = pywikibot.input(
- 'What summary do you want to use?')
- else:
- options['summary'] = arg[9:]
- elif arg.startswith('-checkcurrent'):
+ opt, _, value = arg.partition(':')
+ option = opt[1:] if opt[0] == '-' else None
+ if option == 'summary':
+ options[option] = value or pywikibot.input(
+ 'What summary do you want to use?')
+ elif option == 'checkcurrent':
checkcurrent = True
- elif arg == '-always':
- options['always'] = True
+ elif option == 'always':
+ options[option] = True
else:
genFactory.handleArg(arg)
--
To view, visit
https://gerrit.wikimedia.org/r/507352
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: I343e59706266968894e854c99e7783c6e0a321cc
Gerrit-Change-Number: 507352
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki(a)aol.com>
Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz>
Gerrit-Reviewer: Framawiki <framawiki(a)tools.wmflabs.org>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot (75)