Dzahn has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/375922 )
Change subject: Gerrit: Set base url for commitlink
..
Gerrit: Set base url for commitlink
This is needed so that these will work in polygerrit.
Change-Id: I9d572ab932fb811c2616bc8db3142f298acf3a3b
---
M modules/gerrit/templates/gerrit.config.erb
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Paladox: Looks good to me, but someone else must approve
Chad: Looks good to me, but someone else must approve
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/gerrit/templates/gerrit.config.erb
b/modules/gerrit/templates/gerrit.config.erb
index 92881fc..95cbce9 100644
--- a/modules/gerrit/templates/gerrit.config.erb
+++ b/modules/gerrit/templates/gerrit.config.erb
@@ -30,7 +30,7 @@
link = https://bugzilla.wikimedia.org/$1
[commentlink "changeid"]
match = \\b(I[0-9a-f]{7,40})\\b
-link = "#/q/$1"
+link = "<%= @url %>/#/q/$1"
[commentlink "ci-test-result"]
match = ([^ ]+) (https://integration\\.wikimedia\\.org/ci/[a-zA-Z0-9/_.-]+\";
target=\"_blank\" rel=\"nofollow\">)[^<]+ : ([a-zA-Z_]+)([^<]*)
html = $2$1 $3$4
@@ -39,7 +39,7 @@
link = https://phabricator.wikimedia.org/rSVN$1
[commentlink "commit"]
match = (^|\\s|[:,!?\\(\\[\\{])([0-9a-f]{7,40})\\b
-html = "$1$2"
+html = "$1/#/q/$2\">$2"
[commentlink "cve"]
match = \\b(CVE\\-\\d{4}\\-\\d+)\\b
link = https://cve.mitre.org/cgi-bin/cvename.cgi?name=$1
--
To view, visit https://gerrit.wikimedia.org/r/375922
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9d572ab932fb811c2616bc8db3142f298acf3a3b
Gerrit-PatchSet: 6
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox
Gerrit-Reviewer: Chad
Gerrit-Reviewer: Dzahn
Gerrit-Reviewer: Hashar
Gerrit-Reviewer: Paladox
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits