jenkins-bot has submitted this change and it was merged.
Change subject: [FIX] replace: Allow site to be given positionally
......................................................................
[FIX] replace: Allow site to be given positionally
With 5d832536 it is required that the site parameter is given as a keyword
argument but the script itself actually doesn't use a keyword argument for
that. This now allows the parameter to be given positionally in case subclasses
still provide the parameter positionally.
Bug: T101185
Change-Id: I05200a806309c74677784ec035e599f83c945632
---
M scripts/replace.py
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Ricordisamoa: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/replace.py b/scripts/replace.py
index cf131da..f5eeeb6 100755
--- a/scripts/replace.py
+++ b/scripts/replace.py
@@ -409,7 +409,7 @@
@deprecated_args(acceptall='always')
def __init__(self, generator, replacements, exceptions={},
always=False, allowoverlap=False, recursive=False,
- addedCat=None, sleep=None, summary='', **kwargs):
+ addedCat=None, sleep=None, summary='', site=None, **kwargs):
"""
Constructor.
@@ -450,6 +450,7 @@
"""
super(ReplaceRobot, self).__init__(generator=generator,
always=always,
+ site=site,
**kwargs)
for i, replacement in enumerate(replacements):
--
To view, visit https://gerrit.wikimedia.org/r/215536
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I05200a806309c74677784ec035e599f83c945632
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: Ricordisamoa <ricordisamoa(a)openmailbox.org>
Gerrit-Reviewer: jenkins-bot <>