doc: Corrected Typos in Documentation #20836

pull ppyordanov wants to merge 2 commits into bitcoin:master from ppyordanov:doc-fix-typos changing 18 files +21 −21
  1. ppyordanov commented at 7:41 am on January 3, 2021: contributor

    Scanned the whole source code for typing errors and corrected. Added a translation correction.

    If you’re going to fix typos, can you please fix all of them as seen in the spelling linter script test/lint/lint-spelling.sh, e.g. I believe:

    src/core_read.cpp:131: presense ==> presence src/net_processing.h:68: anounce ==> announce src/netaddress.h:486: compatiblity ==> compatibility src/wallet/walletdb.cpp:429: Crypted ==> Encrypted src/test/validation_tests.cpp:78: excercise ==> exercise test/functional/feature_nulldummy.py:63: unnecssary ==> unnecessary

    and update test/lint/lint-spelling.ignore-words.txt to ignore all of the current false positives.

  2. fanquake commented at 7:53 am on January 3, 2021: member
    Don’t modify upstream files, generated files, historical release notes, subtrees or any translations. Remove any @ mentions from the PR body.
  3. DrahtBot added the label Build system on Jan 3, 2021
  4. DrahtBot added the label Docs on Jan 3, 2021
  5. DrahtBot added the label GUI on Jan 3, 2021
  6. DrahtBot added the label P2P on Jan 3, 2021
  7. DrahtBot added the label UTXO Db and Indexes on Jan 3, 2021
  8. DrahtBot added the label Wallet on Jan 3, 2021
  9. michaelfolkson commented at 10:58 am on January 3, 2021: contributor

    Note also there is already this PR in progress of review. #20762

    There may well be some to-dos for a follow up PR though.

  10. jonatack commented at 11:07 am on January 3, 2021: member
    Concept ACK for any remaining ones after doing #20836 (comment) and #20762 – looks like you found some.
  11. michaelfolkson commented at 11:24 am on January 3, 2021: contributor
    Noisy labels too. A maintainer needs to reduce the Labels to just Docs
  12. DrahtBot commented at 11:27 am on January 3, 2021: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #20811 (refactor: move net_processing implementation details out of header by ajtowns)

    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.

  13. fanquake removed the label Build system on Jan 3, 2021
  14. fanquake removed the label GUI on Jan 3, 2021
  15. fanquake removed the label P2P on Jan 3, 2021
  16. fanquake removed the label UTXO Db and Indexes on Jan 3, 2021
  17. fanquake removed the label Wallet on Jan 3, 2021
  18. hebasto commented at 12:08 pm on January 3, 2021: member
  19. ppyordanov closed this on Jan 3, 2021

  20. ppyordanov force-pushed on Jan 3, 2021
  21. doc: Addressing review comments, excluded:
    generated files
    historical release notes
    subtrees
    translations.
    6d0bc5e410
  22. ppyordanov commented at 5:09 pm on January 3, 2021: contributor

    Addressing review comments, excluded:

    generated files historical release notes subtrees translations.

  23. ppyordanov reopened this on Jan 3, 2021

  24. doc: Reverting subtree file. abc59b5ad0
  25. fanquake commented at 4:37 am on January 4, 2021: member
    Thanks for your contribution. However to move things along I’m going to fix this up and consolidate it with some other changes. Your changes will still be attributed to you.
  26. fanquake closed this on Jan 4, 2021

  27. MarcoFalke referenced this in commit 34322b7f5c on Jan 5, 2021
  28. fanquake locked this on Jan 6, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me