From b08bc1c56ca0dffdaee5c64cf3ef6bc077881dea Mon Sep 17 00:00:00 2001 From: fanquake Date: Sat, 31 Aug 2019 14:18:05 +0800 Subject: [PATCH] Merge #16767: test: Check for codespell in lint-spelling.sh 8340763dc3278c828ee7d525de4c71b4615d17cc Check for codespell in lint-spelling.sh (Kristaps Kaupe) Pull request description: Similar check for `spellcheck` already exists in `lint-shell.sh` ACKs for top commit: practicalswift: ACK 8340763dc3278c828ee7d525de4c71b4615d17cc -- diff looks correct fanquake: ACK 8340763dc3278c828ee7d525de4c71b4615d17cc - Test is the same as in `lint-shell.sh`. Tree-SHA512: f590fc4eae391c5246e86cb70cf79431bd3763ac872f82d3b2222c3a095235ca046a7072c5e61bbaeb3ed6ecd5ff85979769ee8b585de4fb97ac8cfbbf73b956 --- test/lint/lint-spelling.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test/lint/lint-spelling.sh b/test/lint/lint-spelling.sh index 9ab636dd83..59803e7455 100755 --- a/test/lint/lint-spelling.sh +++ b/test/lint/lint-spelling.sh @@ -9,6 +9,11 @@ export LC_ALL=C +if ! command -v codespell > /dev/null; then + echo "Skipping spell check linting since codespell is not installed." + exit 0 +fi + IGNORE_WORDS_FILE=test/lint/lint-spelling.ignore-words.txt if ! codespell --check-filenames --disable-colors --quiet-level=7 --ignore-words=${IGNORE_WORDS_FILE} $(git ls-files -- ":(exclude)build-aux/m4/" ":(exclude)contrib/seeds/*.txt" ":(exclude)depends/" ":(exclude)doc/release-notes/" ":(exclude)src/crypto/" ":(exclude)src/leveldb/" ":(exclude)src/qt/locale/" ":(exclude)src/secp256k1/" ":(exclude)src/univalue/"); then echo "^ Warning: codespell identified likely spelling errors. Any false positives? Add them to the list of ignored words in ${IGNORE_WORDS_FILE}"