jenkins-bot has submitted this change and it was merged.
Change subject: Upgrade to working version of grunt-banana-checker
......................................................................
Upgrade to working version of grunt-banana-checker
0.2.0 is totally broken.
See <https://github.com/wikimedia/grunt-banana-checker/pull/14>.
Change-Id: I58ffe0ebb311f527c52baa2f92b3449b2c618f3b
---
M package.json
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
John Vandenberg: Looks good to me, approved
jenkins-bot: Verified
diff --git a/package.json b/package.json
index 7fa8e36..5387cac 100644
--- a/package.json
+++ b/package.json
@@ -12,7 +12,7 @@
"devDependencies": {
"grunt": "0.4.2",
"grunt-cli": "0.1.13",
- "grunt-banana-checker": "0.2.0"
+ "grunt-banana-checker": "0.2.2"
}
}
--
To view, visit https://gerrit.wikimedia.org/r/234940
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I58ffe0ebb311f527c52baa2f92b3449b2c618f3b
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/i18n
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipedia(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: jenkins-bot <>
jenkins-bot has submitted this change and it was merged.
Change subject: [IMPROV] version: Import at top and BytesIO
......................................................................
[IMPROV] version: Import at top and BytesIO
This moves the imports at the top of the file as suggested by PEP8. It
also uses BytesIO and not StringIO and delegates the decoding to the
underlying XML parser who can interpret the the header to determine the
encoding.
Change-Id: Ib7779b8d26a62dd11f3fcf344ca400239dbbf3f5
---
M pywikibot/version.py
1 file changed, 4 insertions(+), 4 deletions(-)
Approvals:
John Vandenberg: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/version.py b/pywikibot/version.py
index 8de383e..6022dff 100644
--- a/pywikibot/version.py
+++ b/pywikibot/version.py
@@ -18,6 +18,9 @@
import datetime
import subprocess
import codecs
+import xml.dom.minidom
+
+from io import BytesIO
from warnings import warn
@@ -182,8 +185,6 @@
@return: the git hash
@rtype: str
"""
- from io import StringIO
- import xml.dom.minidom
from pywikibot.comms import http
uri = 'https://github.com/wikimedia/%s/!svn/vcc/default' % tag
@@ -192,9 +193,8 @@
"<propfind xmlns=\"DAV:\"><allprop/></propfind>",
headers={'label': str(rev),
'user-agent': 'SVN/1.7.5 {pwb}'})
- data = request.content
- dom = xml.dom.minidom.parse(StringIO(data))
+ dom = xml.dom.minidom.parse(BytesIO(request.raw))
hsh = dom.getElementsByTagName("C:git-commit")[0].firstChild.nodeValue
date = dom.getElementsByTagName("S:date")[0].firstChild.nodeValue
date = time.strptime(date[:19], '%Y-%m-%dT%H:%M:%S')
--
To view, visit https://gerrit.wikimedia.org/r/200256
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib7779b8d26a62dd11f3fcf344ca400239dbbf3f5
Gerrit-PatchSet: 3
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: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: jenkins-bot <>