Skip to content

Commit

Permalink
cmd/gopherbot: match just the whole word document
Browse files Browse the repository at this point in the history
When the title contains the word document then
repeat the check with a regex word boundaries rule then
return positive just in case of a whole word match.

Updates golang/go#31153
  • Loading branch information
Ivan Palladino committed Jun 15, 2019
1 parent ea2aec8 commit 8247ae2
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions cmd/gopherbot/gopherbot.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import (
"net/http"
"os"
"path/filepath"
"regexp"
"sort"
"strconv"
"strings"
Expand Down Expand Up @@ -1988,8 +1989,11 @@ func isDocumentationTitle(t string) bool {
if strings.Contains(t, "godoc:") { // in x/tools, or the dozen places people file it as
return false
}
return strings.Contains(t, "documentation") ||
strings.Contains(t, "docs ")
if strings.Contains(t, "docs ") {
return true
}
return strings.Contains(t, "document") &&
regexp.MustCompile(`\bdocument\b`).MatchString(t)
}

func isGoplsTitle(t string) bool {
Expand Down

0 comments on commit 8247ae2

Please sign in to comment.