doc - fix typos and spellings #32752

pull AdrienUfferte wants to merge 1 commits into bitcoin:master from AdrienUfferte:typo-and-spelling-fixes changing 10 files +12 −12
  1. AdrienUfferte commented at 2:12 pm on June 15, 2025: none

    This PR only fixes typos and spelling error. It does not change any functionality.

    I’m just trying to do a very simple first PR, while reading and familiarizing with the code. I hope I’m not wasting anyone’s time, sorry if this is the case.

  2. DrahtBot commented at 2:12 pm on June 15, 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/32752.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  3. AdrienUfferte force-pushed on Jun 15, 2025
  4. DrahtBot added the label CI failed on Jun 15, 2025
  5. DrahtBot commented at 2:21 pm on June 15, 2025: contributor

    🚧 At least one of the CI tasks failed. Task lint: https://github.com/bitcoin/bitcoin/runs/44125887980 LLM reason (✨ experimental): The CI failure is caused by errors in the lint step due to the subtree directory being touched without a subtree merge.

    Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

  6. AdrienUfferte commented at 2:22 pm on June 15, 2025: none
    Force-pushed a correction asked by the linter
  7. l0rinc commented at 2:36 pm on June 15, 2025: contributor

    Libsecp is a dependency, you have to fix those upstream.

    And before doing cheap typo fixes, we often appreciate if you familiarize yourself with the code by doing code reviews and reproducers instead (running the new tests, benchmarks, checking for errors in other people’s work). Try to help out by validating existing work instead of giving us more.

  8. Fix typos and spellings b1e27a181f
  9. AdrienUfferte force-pushed on Jun 15, 2025
  10. AdrienUfferte commented at 2:44 pm on June 15, 2025: none

    Libsecp is a dependency, you have to fix those upstream.

    And before doing cheap typo fixes, we often appreciate if you familiarize yourself with the code by doing code reviews and reproducers instead (running the new tests, benchmarks, checking for errors in other people’s work). Try to help out by validating existing work instead of giving us more.

    Ho ok, I will try. I will look for a PR where I can find something to say :)

  11. maflcko commented at 7:36 am on June 16, 2025: member
    thx, but this is the wrong repo. You’ll have to submit them upstream
  12. maflcko closed this on Jun 16, 2025


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: 2025-06-17 18:13 UTC

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