cmd/gopherbot: relaxed matching rule for the documentation label

[codebien]

Documentation word is more specific than the document word to match issues related to Documentation.

Fixes golang/go#31153

[gopherbot]

This PR (HEAD: ea2aec8) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/build/+/182419 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

[gopherbot]

Message from Brad Fitzpatrick:

Patch Set 1: Code-Review-1

We still want gopherbot to add Documentation on titles like:

net/http: document the rules around xxxxx

So this patch as-is isn't correct. If you want to not match "textDocument" as in the bug, then the match should be more like /\bdocument\b/. (But for performance reasons, don't use regexps for word boundary detection unless the "document" substring first matches)

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!

[gopherbot]

This PR (HEAD: 8247ae2) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/build/+/182419 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

[gopherbot]

Message from Ivan Palladino:

Patch Set 2:

Patch Set 1: Code-Review-1

We still want gopherbot to add Documentation on titles like:

net/http: document the rules around xxxxx

So this patch as-is isn't correct. If you want to not match "textDocument" as in the bug, then the match should be more like /\bdocument\b/. (But for performance reasons, don't use regexps for word boundary detection unless the "document" substring first matches)

Done

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!

[gopherbot]

Message from Andrew Bonventre:

Patch Set 2:

Can you add a test or update an existing one? Thanks.

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!

[gopherbot]

Message from Brad Fitzpatrick:

Patch Set 1: Code-Review-1

We still want gopherbot to add Documentation on titles like:

net/http: document the rules around xxxxx

So this patch as-is isn't correct. If you want to not match "textDocument" as in the bug, then the match should be more like /\bdocument\b/. (But for performance reasons, don't use regexps for word boundary detection unless the "document" substring first matches)

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!

[gopherbot]

Message from Ivan Palladino:

Patch Set 2:

Patch Set 1: Code-Review-1

We still want gopherbot to add Documentation on titles like:

net/http: document the rules around xxxxx

So this patch as-is isn't correct. If you want to not match "textDocument" as in the bug, then the match should be more like /\bdocument\b/. (But for performance reasons, don't use regexps for word boundary detection unless the "document" substring first matches)

Done

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!

[gopherbot]

Message from Andrew Bonventre:

Patch Set 2:

Can you add a test or update an existing one? Thanks.

---

Please don’t reply on this GitHub thread. Visit golang.org/cl/182419.
After addressing review feedback, remember to publish your drafts!