Skip to content

Conversation

@AdamuAbba
Copy link
Contributor

closes #358

Description

Rewords the doc comment on CreateWithPersistError::Descriptor
(loaded changeset -> provided descriptor(s))

Checklists

All Submissions:

Copy link
Member

@luisschwab luisschwab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5ba59ad

@ValuedMammal ValuedMammal moved this to Needs Review in BDK Wallet Feb 9, 2026
@ValuedMammal ValuedMammal added this to the Wallet 3.0.0 milestone Feb 9, 2026
@ValuedMammal
Copy link
Collaborator

Thank you @AdamuAbba

Copy link
Collaborator

@ValuedMammal ValuedMammal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5ba59ad

ValuedMammal added a commit that referenced this pull request Feb 9, 2026
… comment

5ba59ad fix(doc): reword `CreateWithPersistError::Descriptor` doc comment (shytypes)

Pull request description:

  <!-- You can erase any parts of this template not applicable to your Pull Request. -->
  closes #358

  ### Description

  <!-- Describe the purpose of this PR, what's being adding and/or fixed -->
  Rewords the doc comment on `CreateWithPersistError::Descriptor`
  (loaded changeset -> provided descriptor(s))

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `just p` before pushing

ACKs for top commit:
  luisschwab:
    ACK 5ba59ad
  ValuedMammal:
    ACK 5ba59ad

Tree-SHA512: 55b9d86b5eca8a6c254f6dc74fe7c19d28cfff914248b13c51d19427d520db8247c54d2af54433008bae6b767dc883d092b9ddbbb4eb9f96d7aac56871b8d9bd
@ValuedMammal
Copy link
Collaborator

Somehow github failed to recognize the merge. The commit 5ba59ad has landed on master AFAICT, but let me know if something doesn't look it right.

@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Wallet Feb 10, 2026
@luisschwab
Copy link
Member

I guess GitHub wen't down just as you merged this and things got messed up in their DB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

docs: confusing docs for Descriptor variant of CreateWithPersistError ?

3 participants