[snowpatch] [PATCH 5/8] Don't link to job console output, link to job instead

[Andrew Donnellan]

On 20/07/18 21:57, Russell Currey wrote:
> This is useful for projects with multiple artifacts, the console
> text isn't always the best choice and it's only a click away.
> 
> Signed-off-by: Russell Currey <ruscur at russell.cc>

This seems sensible.

Reviewed-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

> ---
>   src/jenkins.rs | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/jenkins.rs b/src/jenkins.rs
> index 987fae9..b751f59 100644
> --- a/src/jenkins.rs
> +++ b/src/jenkins.rs
> @@ -184,7 +184,7 @@ impl JenkinsBackend {
>       pub fn get_results_url(&self, build_url: &str, job: &BTreeMap<String, String>) -> String {
>           match job.get("artifact") {
>               Some(artifact) => format!("{}/artifact/{}", build_url, artifact),
> -            None => format!("{}/consoleText/", build_url),
> +            None => format!("{}/", build_url),
>           }
>       }
>   
> 

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan at au1.ibm.com  IBM Australia Limited