docs: fix grammatical typos in documentation files #32036

pull VolodymyrBg wants to merge 10 commits into bitcoin:master from VolodymyrBg:Amountoftypos changing 6 files +6 −6

VolodymyrBg commented at 8:57 PM on March 11, 2025: none

Fixed errors and inaccuracies in the project's textual materials. Improved readability of texts and documentation.
Update assumeutxo.md 231339dc18
Update mempool-limits.md 58449758dc
Update release-notes-0.10.0.md 62a11008be
Update build-windows-msvc.md d4fb9fbcff
Update developer-notes.md 4109bd0cfe
Update external-signer.md 7753c3e762
Update productivity.md 37b41643de
Update release-process.md 6984773421
DrahtBot commented at 8:57 PM on March 11, 2025: contributor



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



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32036.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
DrahtBot added the label Docs on Mar 11, 2025
DrahtBot commented at 8:57 PM on March 11, 2025: contributor

📁 Archived release notes are archived and should not be modified.
Update assumeutxo.md 69ec5c7687
Update release-notes-0.10.0.md 3aa481616b
l0rinc commented at 9:02 PM on March 11, 2025: contributor

Nack, misses context completely
in doc/build-windows-msvc.md:82 in 3aa481616b
```
  81 | @@ -82,7 +82,7 @@ ctest --test-dir build --build-config Release  # Use "-j N" for N parallel tests
  82 |  
```
aiswaryasankar commented at 12:52 AM on March 12, 2025:

No new bugs were introduced by the user's changes.

in doc/developer-notes.md:1159 in 3aa481616b

1155 | @@ -1156,7 +1156,7 @@ namespace {
1156 |    `#include "primitives/transactions.h"` quote syntax.
1157 |  
1158 |    - *Rationale*: Bracket syntax is less ambiguous because the preprocessor
1159 | -    searches a fixed list of include directories without taking location of the
1160 | +    searches a fixed list of included directories without taking location of the

aiswaryasankar commented at 12:52 AM on March 12, 2025:

The change from 'include directories' to 'included directories' is incorrect and introduces a grammatical error. The correct term is 'include directories'.

in doc/release-process.md:50 in 3aa481616b

  46 | @@ -47,7 +47,7 @@ Release Process
  47 |    - `m_assumeutxo_data` array should be appended to with the values returned by calling `bitcoin-cli -rpcclienttimeout=0 -named dumptxoutset utxo.dat rollback=<height or hash>`
  48 |      The same height considerations for `defaultAssumeValid` apply.
  49 |  * Consider updating the headers synchronization tuning parameters to account for the chainparams updates.
  50 | -  The optimal values change very slowly, so this isn't strictly necessary every release, but doing so doesn't hurt.
  51 | +  The optimal values change very slowly, so this isn't strictly necessary for every release, but doing so doesn't hurt.

yancyribbens commented at 1:17 AM on March 12, 2025:

This change looks valid

fanquake closed this on Mar 12, 2025
VolodymyrBg deleted the branch on Mar 13, 2025

Contributors

Labels

docs: fix grammatical typos in documentation files #32036

Code Coverage & Benchmarks

Reviews