jenkins-bot has submitted this change and it was merged.

Change subject: add_text: 'page' is now a mandatory argument
......................................................................


add_text: 'page' is now a mandatory argument

to fix SyntaxError: non-default argument follows default argument
https://travis-ci.org/wikimedia/pywikibot-core/jobs/31764485#L871

actually, there was no reason for making 'page' optional

Change-Id: I3de824dfd61276a3943b543db5d86f3ba3ce6b28
---
M scripts/add_text.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  John Vandenberg: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/scripts/add_text.py b/scripts/add_text.py
index cd12b4b..ece394b 100644
--- a/scripts/add_text.py
+++ b/scripts/add_text.py
@@ -117,7 +117,7 @@
 ]
 
 
-def add_text(page=None, addText, summary=None, regexSkip=None,
+def add_text(page, addText, summary=None, regexSkip=None,
              regexSkipUrl=None, always=False, up=False, putText=True,
              oldTextGiven=None, reorderEnabled=True, create=False):
     site = page.site

-- 
To view, visit https://gerrit.wikimedia.org/r/151993
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3de824dfd61276a3943b543db5d86f3ba3ce6b28
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Ricordisamoa <[email protected]>
Gerrit-Reviewer: John Vandenberg <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
Pywikibot-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-commits

Reply via email to