Skip to content
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.

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

doc: correct function name in AssumeUTXO design docs #29518

Closed
wants to merge 1 commit into from

Conversation

jrakibi
Copy link
Contributor

@jrakibi jrakibi commented Feb 29, 2024

Corrected the function name from CompleteSnapshotValidation() to MaybeCompleteSnapshotValidation() in the assumeutxo design documentation.
This change ensures that the documentation accurately reflects the actual function name used in the code

@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 29, 2024

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

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK Empact

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@DrahtBot DrahtBot added the Docs label Feb 29, 2024
@Empact
Copy link
Member

Empact commented Feb 29, 2024

ACK efb70cd

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 1, 2024

🐙 This pull request conflicts with the target branch and needs rebase.

@fanquake
Copy link
Member

fanquake commented Mar 1, 2024

Github is being broken. This has been merged.

@fanquake fanquake closed this Mar 1, 2024
fanquake added a commit to bitcoin-core/gui that referenced this pull request Mar 1, 2024
… design docs

efb70cd doc: correct function name in AssumeUTXO design docs (jrakibi)

Pull request description:

  Corrected the function name from `CompleteSnapshotValidation()` to `MaybeCompleteSnapshotValidation()` in the assumeutxo design documentation.
  This change ensures that the documentation accurately reflects the actual function name used in the code

ACKs for top commit:
  Empact:
    ACK bitcoin/bitcoin@efb70cd

Tree-SHA512: 68b9be3ba710d91a2a955189e227f86b46ccb6a2a13c345d46f276cec6ff12b77ebf9814c4bcb00db7c17e221510e4a2e71175c78a6faf0e0e3159c761bc9b94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants