2019-01-28 18:32:30 -08:00
|
|
|
#!/usr/bin/env bash
|
|
|
|
#
|
|
|
|
# Project nits enforced here
|
|
|
|
#
|
|
|
|
set -e
|
|
|
|
|
|
|
|
cd "$(dirname "$0")/.."
|
|
|
|
source ci/_
|
|
|
|
|
2019-02-09 09:13:35 -08:00
|
|
|
# Logging hygiene: Please don't print from --lib, use the `log` crate instead
|
2019-01-28 18:32:30 -08:00
|
|
|
declare prints=(
|
|
|
|
'print!'
|
|
|
|
'println!'
|
|
|
|
'eprint!'
|
|
|
|
'eprintln!'
|
|
|
|
)
|
|
|
|
|
2019-03-05 13:24:40 -08:00
|
|
|
# Parts of the tree that are expected to be print free
|
|
|
|
declare print_free_tree=(
|
|
|
|
'core/src'
|
2019-05-03 11:01:35 -07:00
|
|
|
'drone/src'
|
|
|
|
'metrics/src'
|
|
|
|
'netutil/src'
|
|
|
|
'runtime/src'
|
|
|
|
'sdk/src'
|
2019-03-05 13:24:40 -08:00
|
|
|
)
|
|
|
|
|
2019-05-03 11:01:35 -07:00
|
|
|
if _ git grep --max-depth=0 "${prints[@]/#/-e }" -- "${print_free_tree[@]}"; then
|
2019-01-28 18:32:30 -08:00
|
|
|
exit 1
|
|
|
|
fi
|
2019-02-09 09:13:35 -08:00
|
|
|
|
|
|
|
|
|
|
|
# Code readability: please be explicit about the type instead of using
|
|
|
|
# Default::default()
|
|
|
|
#
|
|
|
|
# Ref: https://github.com/solana-labs/solana/issues/2630
|
|
|
|
if _ git grep 'Default::default()' -- '*.rs'; then
|
|
|
|
exit 1
|
|
|
|
fi
|
|
|
|
|
2019-05-08 12:15:05 -07:00
|
|
|
# Let's keep a .gitignore for every crate, ensure it's got
|
|
|
|
# /target/ in it
|
|
|
|
declare gitignores_ok=true
|
|
|
|
for i in $(git ls-files \*/Cargo.toml ); do
|
|
|
|
dir=$(dirname "$i")
|
|
|
|
if [[ ! -f $dir/.gitignore ]]; then
|
|
|
|
echo 'error: nits.sh .gitnore missing for crate '"$dir" >&2
|
|
|
|
gitignores_ok=false
|
|
|
|
elif ! grep -q -e '^/target/$' "$dir"/.gitignore; then
|
|
|
|
echo 'error: nits.sh "/target/" apparently missing from '"$dir"'/.gitignore' >&2
|
|
|
|
gitignores_ok=false
|
|
|
|
fi
|
|
|
|
done
|
|
|
|
"$gitignores_ok"
|