Fix build link in PR failure reports (#513)

zware · web-flow · commit cff4b6ae1917 · 2024-08-16T11:02:54.000-05:00
diff --git a/master/custom/pr_reporter.py b/master/custom/pr_reporter.py
@@ -171,8 +171,8 @@ def sendMessage(self, reports):
             )
 
     def _getURLForBuild(self, builderid, build_number):
-        prefix = self.master.config.buildbotURL
-        return prefix + "#builders/%d/builds/%d" % (builderid, build_number)
+        prefix = self.master.config.buildbotURL.rstrip('/')
+        return f"{prefix}/#/builders/{builderid}/builds/{build_number}"
 
     def createStatus(
         self,

Original file line number	Diff line number	Diff line change
`@@ -171,8 +171,8 @@ def sendMessage(self, reports):`
`171`	`171`	`)`
`172`	`172`
`173`	`173`	`def _getURLForBuild(self, builderid, build_number):`
`174`		`- prefix = self.master.config.buildbotURL`
`175`		`- return prefix + "#builders/%d/builds/%d" % (builderid, build_number)`
	`174`	`+ prefix = self.master.config.buildbotURL.rstrip('/')`
	`175`	`+ return f"{prefix}/#/builders/{builderid}/builds/{build_number}"`
`176`	`176`
`177`	`177`	`def createStatus(`
`178`	`178`	`self,`