cmd/releasebot: remove obsolete checkDocs check
The check was added for golang/go#34045, back when doc/contrib.html
lived in the main repo and needed to be manually updated before each
major Go release. This is no longer applicable after CL 229483 and
CL 229485, so remove it before the upcoming Go 1.15 release.
This is compatible with future major releases and does not affect
upcoming minor releases of previous major releases.
Fixes golang/go#40476.
Updates golang/go#34045.
Change-Id: Ifc54dce106633e3d6b9ccdb0db4dae2b7434ce93
Reviewed-on: https://go-review.googlesource.com/c/build/+/245477
Reviewed-by: Carlos Amedee <carlos@golang.org>
diff --git a/cmd/releasebot/main.go b/cmd/releasebot/main.go
index 394af82..f94eae3 100644
--- a/cmd/releasebot/main.go
+++ b/cmd/releasebot/main.go
@@ -380,7 +380,6 @@
if !w.Security {
w.checkReleaseBlockers()
}
- w.checkDocs()
}
w.findOrCreateReleaseIssue()
if len(w.Errors) > 0 && !dryRun {
@@ -592,28 +591,6 @@
}
}
-func (w *Work) checkDocs() {
- // Check that the major version is listed on the project page.
- data, err := ioutil.ReadFile(filepath.Join(w.Dir, "gitwork", "doc/contrib.html"))
- if err != nil {
- w.log.Panic(err)
- }
- major := major(w.Version)
- if !strings.Contains(string(data), major) {
- w.logError("doc/contrib.html does not list major version %s", major)
- }
-}
-
-// major takes a go version like "go1.5", "go1.5.1", "go1.5.2", etc.,
-// and returns the corresponding major version like "go1.5".
-func major(v string) string {
- if strings.Count(v, ".") != 2 {
- // No minor component to drop, return as is.
- return v
- }
- return v[:strings.LastIndex(v, ".")]
-}
-
func (w *Work) writeVersion() (changeID string) {
changeID = fmt.Sprintf("I%x", sha1.Sum([]byte(fmt.Sprintf("cmd/release-version-%s", w.Version))))
