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

Change subject: repository: fix hidden code-git for sf.net
......................................................................


repository: fix hidden code-git for sf.net

The order of words in a pipe sequence matters and having code-git after
code is a noop. Change the URL to optionally match when -git is there to
avoid the pitfall.

Change-Id: I3610f434decf914a0a0267a1c8df0ff3d6fd1429
Signed-off-by: Loic Dachary <l...@dachary.org>
---
M FLOSSbot/repository.py
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/FLOSSbot/repository.py b/FLOSSbot/repository.py
index 8bd6ad8..47bb47c 100644
--- a/FLOSSbot/repository.py
+++ b/FLOSSbot/repository.py
@@ -481,7 +481,7 @@
             if r.status_code != requests.codes.ok:
                 return None
             u = re.findall('git clone (git://git.code.sf.net/p/.*/'
-                           '(?:git|code|code-git))', r.text)
+                           '(?:git|code(?:-git)?))', r.text)
             if len(u) == 1:
                 return u[0]
             u = re.findall('hg clone (http://hg.code.sf.net/p/.*/code)',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3610f434decf914a0a0267a1c8df0ff3d6fd1429
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/bots/FLOSSbot
Gerrit-Branch: master
Gerrit-Owner: Dachary <l...@dachary.org>
Gerrit-Reviewer: Dachary <l...@dachary.org>
Gerrit-Reviewer: Dereckson <dereck...@espace-win.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to