зеркало из https://github.com/golang/pkgsite.git
379 строки
11 KiB
Bash
Executable File
379 строки
11 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# Copyright 2019 The Go Authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style
|
|
# license that can be found in the LICENSE file.
|
|
|
|
source devtools/lib.sh || { echo "Are you at repo root?"; exit 1; }
|
|
|
|
GO=go
|
|
|
|
# Support ** in globs, for check_script_hashes.
|
|
shopt -s globstar
|
|
|
|
warnout() {
|
|
while read line; do
|
|
warn "$line"
|
|
done
|
|
}
|
|
|
|
# filter FILES GLOB1 GLOB2 ...
|
|
# returns the files in FILES that match any of the glob patterns.
|
|
filter() {
|
|
local infiles=$1
|
|
shift
|
|
local patterns=$*
|
|
local outfiles=
|
|
|
|
for pat in $patterns; do
|
|
for f in $infiles; do
|
|
if [[ $f == $pat ]]; then
|
|
outfiles="$outfiles $f"
|
|
fi
|
|
done
|
|
done
|
|
echo $outfiles
|
|
}
|
|
|
|
# Return the files that are modified or added.
|
|
# If there are such files in the working directory, whether or not
|
|
# they are staged for commit, use those.
|
|
# Otherwise, use the files changed since the previous commit.
|
|
modified_files() {
|
|
local working=$(working_files)
|
|
if [[ $working != ' ' ]]; then
|
|
echo $working
|
|
elif [[ $(git rev-parse HEAD) = $(git rev-parse master) ]]; then
|
|
echo ""
|
|
else
|
|
diff_files HEAD^
|
|
fi
|
|
}
|
|
|
|
# Return the files in the working directory that have not been added to the commit.
|
|
working_files() {
|
|
echo "$(diff_files '') $(diff_files --cached)"
|
|
}
|
|
|
|
|
|
# Helper for modified_files. It asks git for all modified, added or deleted
|
|
# files, and keeps only the latter two.
|
|
diff_files() {
|
|
git diff --name-status $* | awk '$1 ~ /^R/ { print $3; next } $1 != "D" { print $2 }'
|
|
}
|
|
|
|
# codedirs lists directories that contain discovery code. If they include
|
|
# directories containing external code, those directories must be excluded in
|
|
# findcode below.
|
|
codedirs=(
|
|
"cmd"
|
|
"internal"
|
|
"migrations"
|
|
"static"
|
|
)
|
|
|
|
# verify_header checks that all given files contain the standard header for Go
|
|
# projects.
|
|
verify_header() {
|
|
if [[ "$@" != "" ]]; then
|
|
for FILE in $@
|
|
do
|
|
# Allow for the copyright header to start on either of the first three
|
|
# lines, to accommodate conventions for CSS and HTML.
|
|
line="$(head -4 $FILE)"
|
|
if [[ ! $line == *"The Go Authors. All rights reserved."* ]] &&
|
|
[[ ! $line == "// DO NOT EDIT. This file was copied from" ]]; then
|
|
err "missing license header: $FILE"
|
|
fi
|
|
done
|
|
fi
|
|
}
|
|
|
|
# findcode finds source files in the repo, skipping third-party source.
|
|
findcode() {
|
|
find ${codedirs[@]} \
|
|
-not -path '*/third_party/*' \
|
|
\( -name *.go -o -name *.sql -o -name *.tmpl -o -name *.css -o -name *.js \)
|
|
}
|
|
|
|
# ensure_go_binary verifies that a binary exists in $PATH corresponding to the
|
|
# given go-gettable URI. If no such binary exists, it is fetched via `go install`.
|
|
ensure_go_binary() {
|
|
local binary=$(basename $1)
|
|
if ! [ -x "$(command -v $binary)" ]; then
|
|
info "Installing: $1"
|
|
# Run in a subshell for convenience, so that we don't have to worry about
|
|
# our PWD.
|
|
(set -x; cd && $GO install $1@latest)
|
|
fi
|
|
}
|
|
|
|
# check_headers checks that all source files that have been staged in this
|
|
# commit, and all other non-third-party files in the repo, have a license
|
|
# header.
|
|
check_headers() {
|
|
if [[ $# -gt 0 ]]; then
|
|
info "Checking listed files for license header"
|
|
verify_header $*
|
|
else
|
|
info "Checking staged files for license header"
|
|
# Check code files that have been modified or added.
|
|
verify_header $(git diff --cached --name-status | grep -vE "^D" | cut -f 2- | grep -E ".go$|.sql$|.sh$")
|
|
info "Checking internal files for license header"
|
|
verify_header $(findcode)
|
|
fi
|
|
}
|
|
|
|
# bad_migrations outputs migrations with bad sequence numbers.
|
|
bad_migrations() {
|
|
ls migrations | cut -d _ -f 1 | sort | uniq -c | grep -vE '^\s+2 '
|
|
}
|
|
|
|
# check_bad_migrations looks for sql migration files with bad sequence numbers,
|
|
# possibly resulting from a bad merge.
|
|
check_bad_migrations() {
|
|
info "Checking for bad migrations"
|
|
bad_migrations | while read line
|
|
do
|
|
err "unexpected number of migrations: $line"
|
|
done
|
|
}
|
|
|
|
# check_unparam runs unparam on source files.
|
|
check_unparam() {
|
|
echo "unparam disabled until ssa supports generics"
|
|
# TODO: uncomment when working
|
|
# ensure_go_binary mvdan.cc/unparam
|
|
# runcmd unparam ./...
|
|
}
|
|
|
|
# check_vet runs go vet on source files.
|
|
check_vet() {
|
|
runcmd $GO vet -all ./...
|
|
}
|
|
|
|
# check_staticcheck runs staticcheck on source files.
|
|
check_staticcheck() {
|
|
ensure_go_binary honnef.co/go/tools/cmd/staticcheck
|
|
runcmd staticcheck $(go list ./... | grep -v third_party | grep -v internal/doc | grep -v internal/render)
|
|
}
|
|
|
|
# check_misspell runs misspell on source files.
|
|
check_misspell() {
|
|
ensure_go_binary github.com/client9/misspell/cmd/misspell
|
|
runcmd misspell cmd/**/*.{go,sh} internal/**/* README.md
|
|
}
|
|
|
|
# check_templates runs go-template-lint on template files. Unfortunately it
|
|
# doesn't handler the /helpers/ fileglob correctly, so it is too noisy to be
|
|
# included in standard checks.
|
|
check_templates() {
|
|
ensure_go_binary sourcegraph.com/sourcegraph/go-template-lint
|
|
runcmd go-template-lint \
|
|
-f=internal/frontend/server.go \
|
|
-t=internal/frontend/server.go \
|
|
-td=static | warnout
|
|
}
|
|
|
|
|
|
script_hash_glob='static/**/*.tmpl'
|
|
|
|
# check_script_hashes checks that our CSP hashes match the ones
|
|
# for our HTML scripts.
|
|
check_script_hashes() {
|
|
runcmd $GO run ./devtools/cmd/csphash $script_hash_glob
|
|
}
|
|
|
|
# run_build_static builds JavaScript output from TypeScript source files.
|
|
run_build_static() {
|
|
runcmd $GO run ./devtools/cmd/static
|
|
files=$(working_files)
|
|
if [[ $(filter "$files" 'static/**/*.min*') != '' ]]; then
|
|
err "minimized CSS files are not consistent with unminimized ones; run ./devtools/cmd/static to regenerate them"
|
|
fi
|
|
}
|
|
|
|
run_npm() {
|
|
npmcmd=${GO_DISCOVERY_NPM_CMD:-"./devtools/nodejs.sh npm"}
|
|
# Run npm install if node_modules directory does not exist.
|
|
if [ ! -d "node_modules" ]; then
|
|
runcmd $npmcmd install --quiet
|
|
fi
|
|
runcmd $npmcmd $@
|
|
}
|
|
|
|
run_npx() {
|
|
npxcmd=${GO_DISCOVERY_NPX_CMD:-"./devtools/nodejs.sh npx"}
|
|
# Run npm install if node_modules directory does not exist.
|
|
if [ ! -d "node_modules" ]; then
|
|
run_npm install --quiet
|
|
fi
|
|
runcmd $npxcmd $@
|
|
}
|
|
|
|
prettier_file_globs='**/*.md'
|
|
|
|
# run_prettier runs prettier on CSS, JS, and MD files. Uses globally
|
|
# installed prettier if available or a dockerized installation as a
|
|
# fallback.
|
|
run_prettier() {
|
|
local files=$*
|
|
if [[ $files = '' ]]; then
|
|
files=$prettier_file_globs
|
|
fi
|
|
run_npx prettier --write $files
|
|
}
|
|
|
|
go_linters() {
|
|
check_vet
|
|
check_staticcheck
|
|
check_misspell
|
|
check_unparam
|
|
}
|
|
|
|
standard_linters() {
|
|
run_build_static
|
|
check_headers
|
|
check_bad_migrations
|
|
go_linters
|
|
check_script_hashes
|
|
}
|
|
|
|
|
|
usage() {
|
|
cat <<EOUSAGE
|
|
Usage: $0 [subcommand]
|
|
Available subcommands:
|
|
help - display this help message
|
|
(empty) - run all standard checks and tests
|
|
ci - run checks and tests suitable for continuous integration
|
|
cl - run checks and tests on the current CL, suitable for a commit or pre-push hook
|
|
lint - run all standard linters below:
|
|
headers - (lint) check source files for the license disclaimer
|
|
migrations - (lint) check migration sequence numbers
|
|
misspell - (lint) run misspell on source files
|
|
npm - run npm commands or scripts from package.json
|
|
npx - run a command from a local or remote npm package
|
|
script_hashes - (lint) check script hashes
|
|
script_output - (lint) check script output
|
|
staticcheck - (lint) run staticcheck on source files
|
|
unparam - (lint) run unparam on source files
|
|
prettier - (lint, nonstandard) run prettier on .js and .css files.
|
|
templates - (lint, nonstandard) run go-template-lint on templates
|
|
EOUSAGE
|
|
}
|
|
|
|
# Packages to run without the race detector on CI.
|
|
# (They time out with -race.)
|
|
declare -A no_race
|
|
no_race=(
|
|
[golang.org/x/pkgsite/internal/frontend]=1
|
|
[golang.org/x/pkgsite/internal/worker]=1
|
|
[golang.org/x/pkgsite/internal/testing/integration]=1
|
|
)
|
|
|
|
main() {
|
|
case "$1" in
|
|
"-h" | "--help" | "help")
|
|
usage
|
|
exit 0
|
|
;;
|
|
"")
|
|
standard_linters
|
|
run_prettier
|
|
run_npm run lint -- --fix
|
|
run_npm run test
|
|
runcmd $GO mod tidy
|
|
runcmd env GO_DISCOVERY_TESTDB=true go test ./...
|
|
runcmd $GO test ./internal/secrets
|
|
run_npm audit
|
|
;;
|
|
cl)
|
|
# Similar to the above, but only run checks that apply to files in this commit.
|
|
files=$(modified_files)
|
|
if [[ $files = '' ]]; then
|
|
info "No modified files; nothing to do."
|
|
exit 0
|
|
fi
|
|
info "Running checks on:"
|
|
info " " $files
|
|
|
|
if [[ $(filter "$files" $script_hash_glob) != '' ]]; then
|
|
check_script_hashes
|
|
run_build_static
|
|
fi
|
|
check_headers $(filter "$files" '*.go' '*.sql' '*.sh')
|
|
if [[ $(filter "$files" 'migrations/*') != '' ]]; then
|
|
check_bad_migrations
|
|
fi
|
|
if [[ $(filter "$files" '*.go') != '' ]]; then
|
|
go_linters
|
|
fi
|
|
pfiles=$(filter "$files" $prettier_file_globs)
|
|
if [[ $pfiles != '' ]]; then
|
|
run_prettier $pfiles
|
|
fi
|
|
if [[ $(filter "$files" 'static/**') != '' ]]; then
|
|
run_npm run lint -- --fix
|
|
run_npm run test
|
|
fi
|
|
runcmd $GO mod tidy
|
|
runcmd env GO_DISCOVERY_TESTDB=true go test ./...
|
|
runcmd $GO test ./internal/secrets
|
|
;;
|
|
|
|
ci)
|
|
# Similar to the no-arg mode, but omit actions that require GCP
|
|
# permissions or that don't test the code.
|
|
# Also, run the race detector on most tests.
|
|
local start=`date +%s`
|
|
|
|
# Explicitly mark the working directory as safe in CI.
|
|
# https://github.com/docker-library/golang/issues/452
|
|
local wd=$(pwd)
|
|
runcmd git config --system --add safe.directory ${wd}
|
|
|
|
standard_linters
|
|
# Print how long it takes to download dependencies and run the standard
|
|
# linters in CI.
|
|
local end=`date +%s`
|
|
echo
|
|
echo "--------------------"
|
|
echo "DONE: $((end-start)) seconds"
|
|
echo "--------------------"
|
|
|
|
for pkg in $($GO list ./...); do
|
|
if [[ ${no_race[$pkg]} = '' ]]; then
|
|
race="$race $pkg"
|
|
fi
|
|
done
|
|
runcmd env GO_DISCOVERY_TESTDB=true $GO test -race -count=1 $race
|
|
runcmd env GO_DISCOVERY_TESTDB=true $GO test -count=1 ${!no_race[*]}
|
|
;;
|
|
lint) standard_linters ;;
|
|
headers) check_headers ;;
|
|
migrations) check_migrations ;;
|
|
misspell) check_misspell ;;
|
|
staticcheck) check_staticcheck ;;
|
|
prettier)
|
|
shift
|
|
run_prettier $*
|
|
;;
|
|
templates) check_templates ;;
|
|
unparam) check_unparam ;;
|
|
script_hashes) check_script_hashes ;;
|
|
build_static) run_build_static ;;
|
|
npm) run_npm ${@:2} ;;
|
|
npx) run_npx ${@:2} ;;
|
|
|
|
*)
|
|
usage
|
|
exit 1
|
|
esac
|
|
if [[ $EXIT_CODE != 0 ]]; then
|
|
err "FAILED; see errors above"
|
|
fi
|
|
exit $EXIT_CODE
|
|
}
|
|
|
|
main $@
|