Skip to content

Commit

Permalink
Merge #16767: test: Check for codespell in lint-spelling.sh
Browse files Browse the repository at this point in the history
210e28c 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 210e28c -- diff looks correct
  fanquake:
    ACK 210e28c  - Test is the same as in `lint-shell.sh`.

Tree-SHA512: f590fc4eae391c5246e86cb70cf79431bd3763ac872f82d3b2222c3a095235ca046a7072c5e61bbaeb3ed6ecd5ff85979769ee8b585de4fb97ac8cfbbf73b956
  • Loading branch information
fanquake committed Aug 31, 2019
2 parents 26cef43 + 210e28c commit 7847231
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions test/lint/lint-spelling.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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/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}"
Expand Down

0 comments on commit 7847231

Please sign in to comment.