script: fix spelling linter raising spuriously on "invokable" #22138

pull jonatack wants to merge 1 commits into bitcoin:master from jonatack:fix-spurious-linter-spelling changing 1 files +1 −0

jonatack commented at 11:58 AM on June 3, 2021: member

"invokable" is a valid word that means to be callable, but the linter is raising on it:

$ test/lint/lint-spelling.sh 
contrib/guix/guix-attest:18: invokable ==> invocable
contrib/guix/guix-clean:18: invokable ==> invocable
contrib/guix/guix-verify:18: invokable ==> invocable
^ Warning: codespell identified likely spelling errors. Any false positives? Add them to the list of ignored words in test/lint/lint-spelling.ignore-words.txt

script: fix spelling linter raising spuriously on "invokable" 8050eb43bf
michaelfolkson commented at 12:02 PM on June 3, 2021: contributor

Concept ACK. I think this should be doc: in the title rather than script:?
jonatack renamed this:
~~script: fix spelling linter raising spuriously on "invokable"~~
doc: fix spelling linter raising spuriously on "invokable"
on Jun 3, 2021
jonatack commented at 12:08 PM on June 3, 2021: member

Changed to doc though I think it should be script (the linter script) or test.
jonatack renamed this:
~~doc: fix spelling linter raising spuriously on "invokable"~~
script: fix spelling linter raising spuriously on "invokable"
on Jun 3, 2021
jonatack commented at 12:09 PM on June 3, 2021: member

Back to script :smile:
michaelfolkson commented at 12:11 PM on June 3, 2021: contributor

You are right, sorry :)

I thought script was a prefix for changes to Bitcoin script but appears not! That would be consensus

script for changes to the scripts and tools https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#creating-the-pull-request
DrahtBot added the label Tests on Jun 3, 2021
MarcoFalke commented at 12:35 PM on June 3, 2021: member

cr ACK 8050eb43bf15501e33ec5312918d926e47e4fc8d
MarcoFalke merged this on Jun 3, 2021
MarcoFalke closed this on Jun 3, 2021
jonatack deleted the branch on Jun 3, 2021
sidhujag referenced this in commit 72c72492cc on Jun 3, 2021
gwillen referenced this in commit d28106be65 on Jun 1, 2022
DrahtBot locked this on Aug 16, 2022

Contributors

Labels