Skip to content

doc: fix typos #31253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
threewebcode wants to merge 2 commits into from
Closed

doc: fix typos #31253

threewebcode wants to merge 2 commits into from

Conversation

@threewebcode
Copy link
Contributor

@threewebcode threewebcode commented Nov 7, 2024 •
edited
Loading

There are some typos in the code comments and use this change to update them

@DrahtBot
Copy link
Contributor

DrahtBot commented Nov 7, 2024 •
edited
Loading

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/31253.

Reviews

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

@maflcko
Copy link
Member

maflcko commented Nov 8, 2024

Please squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits

@fanquake fanquake changed the title fix: recitfy typos doc: recitfy typos Nov 8, 2024
@DrahtBot DrahtBot added the Docs label Nov 8, 2024
@rkrux
Copy link
Contributor

rkrux commented Nov 8, 2024

If your change contains a merge commit, the above workflow may not work and you will need to remove the merge commit first. See the next section for details on how to rebase.

The merge commit would need to be removed: https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#rebasing-changes

doc: recitfy typos

Also, there's a typo in the PR title. :)

@dergoegge
Copy link
Member

dergoegge commented Nov 8, 2024

Same as #30259?

@threewebcode threewebcode changed the title doc: recitfy typos doc: fix typos Nov 11, 2024
@fanquake
Copy link
Member

fanquake commented Nov 11, 2024

Pulled this into another branch.

@fanquake fanquake closed this Nov 11, 2024
@fanquake fanquake mentioned this pull request Nov 11, 2024
Merged
fanquake added a commit that referenced this pull request Nov 12, 2024
@fanquake
Merge #31271: doc: correct typos
3fb6229 
726cbee doc: correct typos (fanquake)
9fdfb73 doc: fix typos (Afanti)

Pull request description:

  Includes #31253.
  Includes #31239 (review).
  Fixes remaining lint output.

ACKs for top commit:
  l0rinc:
    ACK 726cbee
  rkrux:
    crACK 726cbee
  tdb3:
    ACK 726cbee

Tree-SHA512: 51978343f11fb5f0c6b824d92dbfc9999952373a9f790ab79ef8750f920f1c020c092ca874c9e39f478d12d85cdadcfd8c63dda0cbb02745bc55fda28d371e4c
@bitcoin bitcoin locked and limited conversation to collaborators Nov 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

Docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@threewebcode @DrahtBot @maflcko @rkrux @dergoegge @fanquake