tools: Filter "highlight" case insensitive
What the developer wrote
Authored by Peter Neuroth
tools: Filter "highlight" case insensitive
There was a problem with a ‘highlight’ that was misunderstood as a
spelling mistake in lib-wally. Since ‘hightlight’ is already filtered
out, we simply instruct grep to ignore upper/lower case when filtering.
Signed-off-by: Peter Neuroth <pet.v.ne@gmail.com>
What changed, and why it matters
This commit changes a spelling-check script so it ignores uppercase and lowercase when filtering out the word 'highlight'. It is a build/tooling fix to prevent false positives in the project's spell checker and has no security relevance.
No security action needed. This is a benign tooling fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change modifies tools/check-spelling.sh, adding the -i flag to a grep command that filters out ‘highlighting’ and ‘LightningGrpc’ from a list of possible misspellings of ‘lightning’. This makes the filter case-insensitive, avoiding false positives when ‘Highlight’ appears with different capitalization. There is no code behavior change in Core Lightning itself.
Changed components
tools/check-spelling.shInspect captured patch +1 / −1
diff --git a/tools/check-spelling.sh b/tools/check-spelling.sh
index 68cd2023..a66d0178 100755
--- a/tools/check-spelling.sh
+++ b/tools/check-spelling.sh
@@ -1,6 +1,6 @@
#!/usr/bin/env bash
-if git --no-pager grep -nHiE 'l[ightn]{6}g|l[ightn]{8}g|ilghtning|lgihtning|lihgtning|ligthning|lighnting|lightinng|lightnnig|lightnign' -- . ':!tools/check-spelling.sh' ':!tests/data/routing_gossip_store' | grep -vE "highlighting|LightningGrpc"; then
+if git --no-pager grep -nHiE 'l[ightn]{6}g|l[ightn]{8}g|ilghtning|lgihtning|lihgtning|ligthning|lighnting|lightinng|lightnnig|lightnign' -- . ':!tools/check-spelling.sh' ':!tests/data/routing_gossip_store' | grep -viE "highlighting|LightningGrpc"; then
echo "Identified a likely misspelling of the word \"lightning\" (see above). Please fix."
echo "Is this warning incorrect? Please teach tools/check-spelling.sh about the exciting new word."
exit 1
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.