doc: fix spelling #21875

pull jbampton wants to merge 1 commits into bitcoin:master from jbampton:fix-spelling changing 8 files +9 −10

jbampton commented at 11:42 AM on May 7, 2021: contributor

Fixed some typos
fanquake commented at 11:47 AM on May 7, 2021: member

You cannot fix typos in subtrees, so please remove all changes to them. Feel free to send the changes upstream.
in test/lint/lint-spelling.ignore-words.txt:14 in f14ec80c0f outdated
```
  10 | @@ -11,5 +11,4 @@ nin
  11 |  ser
  12 |  unparseable
  13 |  unser
  14 | -useable
```
maflcko commented at 11:48 AM on May 7, 2021:

https://www.merriam-webster.com/dictionary/useable

Also, if you want to modify libraries, you will have to do it upstream

fanquake commented at 11:49 AM on May 7, 2021:

Not sure why you've dropped this and changed usages to usable, so you can revert that as well.

jbampton commented at 11:53 AM on May 7, 2021:

Most of the spelling in the codebase uses usable not useable.

<img width="1011" alt="Screen Shot 2021-05-07 at 9 51 10 pm" src="https://user-images.githubusercontent.com/418747/117445821-86f6e680-af7e-11eb-8497-6810336abf92.png">
jbampton force-pushed on May 7, 2021
in contrib/guix/guix-clean:18 in b5781b7bf4 outdated
```
  14 | @@ -15,7 +15,7 @@ source "$(dirname "${BASH_SOURCE[0]}")/libexec/prelude.bash"
  15 |  ###################
  16 |  
  17 |  ################
  18 | -# Required non-builtin commands should be invokable
  19 | +# Required non-builtin commands should be invocable
```
flack commented at 1:08 PM on May 7, 2021:

is that really a typo? invokable seems to exist: https://en.wiktionary.org/wiki/invokable (and that site lists invocable as an alternative spelling)

jbampton commented at 1:18 PM on May 7, 2021:

invocable is in the codebase twice and invokable once. Should I change them all to invokable ?
jbampton force-pushed on May 7, 2021
doc: fix spelling 84e19d0673
DrahtBot added the label Build system on May 7, 2021
DrahtBot added the label Scripts and tools on May 7, 2021
jbampton force-pushed on May 7, 2021
jarolrod commented at 2:06 AM on May 10, 2021: member

Concept 0 % s/contruct/construct

Invokable and useable or both acceptable alternatives to invocable and usable. Their usage is correct and you're just swapping out one correct version for another. Bitcoin contributions come from all over the world, no need to impose the 'American' version of words.
laanwj commented at 3:00 PM on May 10, 2021: member

Agree with @jarolrod here. These seem to be okay (apart from "contruct"). Don't make changes just for the sake of making changes, please.
DrahtBot commented at 3:46 AM on May 19, 2021: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #21992 (Feefilter cleanups by amadeuszpawlik)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
maflcko commented at 4:56 AM on May 20, 2021: member

Thank you for your contribution. However, this patch seems controversial, so I am going to close for now.
maflcko closed this on May 20, 2021
bitcoin locked this on Aug 18, 2022
jbampton deleted the branch on Apr 1, 2025

Contributors

Labels

Build system Scripts and tools

doc: fix spelling #21875

Conflicts