Don't spellcheck testdata/ files or the file containing known
misspellings.
Change-Id: Id9b7ff1050a091901904722fc6bdc7bb595b4574
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/595384
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Damien Neil <dneil@google.com>
Restore the check for unused parameters and remove unused parameters
found by the check.
Change-Id: I80fe9b1cab507eeae8eb1f27e2395824330868c5
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/551436
Reviewed-by: Damien Neil <dneil@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Run all checks in "checks.bash" that do not require network access in
short mode (presubmit), instead of skipping all the checks. This will
help us catch more issues before CL submission.
Also fixes the license header check to only look at files, not
directories.
Change-Id: Ia1da9883c1d5cb69aca657f8468255da747119c2
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/534576
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Auto-Submit: Tatiana Bradley <tatianabradley@google.com>
A mention of "github.com/julz/importas" caused the misspell tool to
fail, thinking that "importas" was a misspelling of "imports",
so add an exception.
Change-Id: I970a2556c3ebe0e244aee27650c166cfb925116a
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/523636
Run-TryBot: Tatiana Bradley <tatianabradley@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Tatiana Bradley <tatianabradley@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Add a test that a file in data/ cannot be deleted.
Change-Id: Ie668b75020d7c184ddbb8ae66f612e945dd62e9b
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/439298
Reviewed-by: Tatiana Bradley <tatiana@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Julie Qiu <julie@golang.org>
Factor out the tracing and metrics code from the server
and put it in its own package.
This is a first step towards sharing it with other projects.
Change-Id: I36a04933accc11300f360a410c00a10c8a132dda
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/435470
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Tatiana Bradley <tatiana@golang.org>
Reviewed-by: Julie Qiu <julieqiu@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
Update the exp repo to the latest version. This will give us the new
event metrics code, which we will then use for worker metrics.
Also, add the -compat=1.17 flag to go mod tidy; otherwise it complains
about maintaining 1.16 compatibility, which we don't care about.
Change-Id: I5abb1aecb8b9d6b424223ea00680ea98bd29e4da
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/386696
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Add support for OpenTelemetry tracing of certain
important functions, using the event package.
We don't have a burning need for traces, but they
are nice to have, and this was an important exercise
for validating the approach of the event package.
Change-Id: I37d1f56f06f425f3b1eb885877a0d2f5ac85a098
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/380440
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
all.bash is replaced with the checks.bash setup from x/vuln, so that
tests run on TryBots.
Change-Id: I49f2265343e9e962b8587eb9a733a52651466737
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/373156
Trust: Julie Qiu <julie@golang.org>
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Jonathan Amsterdam <jba@google.com>