From c8cc47ff55092511090a79f6ab0af0952464c812 Mon Sep 17 00:00:00 2001 From: Brad Fitzpatrick Date: Mon, 14 Jan 2019 18:56:35 +0000 Subject: [PATCH] app/appengine: remove the security branches from the build dashboard They're causing unnecessary work and clutter on the build.golang.org page. Fixes golang/go#29513 Change-Id: Ieb9bbe4caf0041dd2a2f0e34b4d52a283de56224 Reviewed-on: https://go-review.googlesource.com/c/157817 Reviewed-by: Dmitri Shuralyov --- app/appengine/dash.go | 14 ++++++++------ app/appengine/ui.go | 4 ++++ 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/app/appengine/dash.go b/app/appengine/dash.go index 5f1397fe..587c371a 100644 --- a/app/appengine/dash.go +++ b/app/appengine/dash.go @@ -204,10 +204,12 @@ var gccgoDash = &Dashboard{ // This also prevents the builder infrastructure // from testing sub-repos against these branches. var hiddenBranches = map[string]bool{ - "release-branch.go1.4": true, - "release-branch.go1.5": true, - "release-branch.go1.6": true, - "release-branch.go1.7": true, - "release-branch.go1.8": true, - "release-branch.go1.9": true, + "release-branch.go1.4": true, + "release-branch.go1.5": true, + "release-branch.go1.6": true, + "release-branch.go1.7": true, + "release-branch.go1.8": true, + "release-branch.go1.9": true, + "release-branch.go1.10-security": true, + "release-branch.go1.11-security": true, } diff --git a/app/appengine/ui.go b/app/appengine/ui.go index 600b0b94..b32037be 100644 --- a/app/appengine/ui.go +++ b/app/appengine/ui.go @@ -184,6 +184,10 @@ func listBranches(c context.Context) (branches []string) { return } for _, c := range commits { + if strings.HasPrefix(c.Branch, "release-branch.go") && + strings.HasSuffix(c.Branch, "-security") { + continue + } branches = append(branches, c.Branch) } return