travis, appveyor, build: add source spell checking
This commit is contained in:
parent
c5df37c111
commit
e4181a7f1b
@ -90,7 +90,7 @@ install:
|
|||||||
- go get golang.org/x/tools/cmd/cover
|
- go get golang.org/x/tools/cmd/cover
|
||||||
script:
|
script:
|
||||||
- go run build/ci.go install
|
- go run build/ci.go install
|
||||||
- go run build/ci.go test -coverage -vet
|
- go run build/ci.go test -coverage -vet -misspell
|
||||||
|
|
||||||
notifications:
|
notifications:
|
||||||
webhooks:
|
webhooks:
|
||||||
|
@ -36,4 +36,4 @@ after_build:
|
|||||||
|
|
||||||
test_script:
|
test_script:
|
||||||
- set CGO_ENABLED=1
|
- set CGO_ENABLED=1
|
||||||
- go run build\ci.go test -vet -coverage
|
- go run build\ci.go test -vet -coverage -misspell
|
||||||
|
25
build/ci.go
25
build/ci.go
@ -24,7 +24,7 @@ Usage: go run ci.go <command> <command flags/arguments>
|
|||||||
Available commands are:
|
Available commands are:
|
||||||
|
|
||||||
install [-arch architecture] [ packages... ] -- builds packages and executables
|
install [-arch architecture] [ packages... ] -- builds packages and executables
|
||||||
test [ -coverage ] [ -vet ] [ packages... ] -- runs the tests
|
test [ -coverage ] [ -vet ] [ -misspell] [ packages... ] -- runs the tests
|
||||||
archive [-arch architecture] [ -type zip|tar ] [ -signer key-envvar ] [ -upload dest ] -- archives build artefacts
|
archive [-arch architecture] [ -type zip|tar ] [ -signer key-envvar ] [ -upload dest ] -- archives build artefacts
|
||||||
importkeys -- imports signing keys from env
|
importkeys -- imports signing keys from env
|
||||||
debsrc [ -signer key-id ] [ -upload dest ] -- creates a debian source package
|
debsrc [ -signer key-id ] [ -upload dest ] -- creates a debian source package
|
||||||
@ -262,6 +262,7 @@ func goToolArch(arch string, subcmd string, args ...string) *exec.Cmd {
|
|||||||
func doTest(cmdline []string) {
|
func doTest(cmdline []string) {
|
||||||
var (
|
var (
|
||||||
vet = flag.Bool("vet", false, "Whether to run go vet")
|
vet = flag.Bool("vet", false, "Whether to run go vet")
|
||||||
|
misspell = flag.Bool("misspell", false, "Whether to run the spell checker")
|
||||||
coverage = flag.Bool("coverage", false, "Whether to record code coverage")
|
coverage = flag.Bool("coverage", false, "Whether to record code coverage")
|
||||||
)
|
)
|
||||||
flag.CommandLine.Parse(cmdline)
|
flag.CommandLine.Parse(cmdline)
|
||||||
@ -287,7 +288,29 @@ func doTest(cmdline []string) {
|
|||||||
if *vet {
|
if *vet {
|
||||||
build.MustRun(goTool("vet", packages...))
|
build.MustRun(goTool("vet", packages...))
|
||||||
}
|
}
|
||||||
|
if *misspell {
|
||||||
|
// The spell checker doesn't work on packages, gather all .go files for it
|
||||||
|
sources := []string{}
|
||||||
|
for _, pkg := range packages {
|
||||||
|
// Gather all the source files of the package
|
||||||
|
out, err := goTool("list", "-f", "{{.Dir}}{{range .GoFiles}}\n{{.}}{{end}}{{range .CgoFiles}}\n{{.}}{{end}}{{range .TestGoFiles}}\n{{.}}{{end}}", pkg).CombinedOutput()
|
||||||
|
if err != nil {
|
||||||
|
log.Fatalf("source file listing failed: %v\n%s", err, string(out))
|
||||||
|
}
|
||||||
|
// Retrieve the folder and assemble the source list
|
||||||
|
lines := strings.Split(string(out), "\n")
|
||||||
|
|
||||||
|
root := lines[0]
|
||||||
|
for _, line := range lines[1:] {
|
||||||
|
if line = strings.TrimSpace(line); line != "" {
|
||||||
|
sources = append(sources, filepath.Join(root, line))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
// Download the spell checker tool and run on all source files
|
||||||
|
build.MustRun(goTool("get", "github.com/client9/misspell/cmd/misspell"))
|
||||||
|
build.MustRunCommand(filepath.Join(GOBIN, "misspell"), append([]string{"-error"}, sources...)...)
|
||||||
|
}
|
||||||
// Run the actual tests.
|
// Run the actual tests.
|
||||||
gotest := goTool("test")
|
gotest := goTool("test")
|
||||||
// Test a single package at a time. CI builders are slow
|
// Test a single package at a time. CI builders are slow
|
||||||
|
Loading…
Reference in New Issue
Block a user