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

fix(persist): add default feature to enable bdk_chain/std #1423

Merged
merged 1 commit into from
May 8, 2024

Conversation

notmandatory
Copy link
Member

Description

This PR adds a default feature to bdk_persist so it can be build on its own. Once #1422 is done we can remove the defaultagain.

Notes to the reviewers

I need to be able to build bdk_persist on its own so I can publish it to crates.io.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@notmandatory notmandatory added the bug Something isn't working label May 2, 2024
@notmandatory notmandatory added this to the 1.0.0-alpha milestone May 2, 2024
@notmandatory notmandatory self-assigned this May 2, 2024
@notmandatory
Copy link
Member Author

nevermind! I was able to publish bdk_persist with this command:

cargo publish --features bdk_chain/std

@ValuedMammal ValuedMammal mentioned this pull request May 4, 2024
7 tasks
@notmandatory notmandatory reopened this May 7, 2024
@notmandatory
Copy link
Member Author

@ValuedMammal per your comment in #1422 I agree, doesn't hurt to get this fixed so docs build in our next alpha. @oleonardolima can revert this change as part of the rust-bitcoin 0.32 update.

Copy link
Contributor

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

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

ACK a577c22

@ValuedMammal
Copy link
Contributor

ACK a577c22

Copy link
Contributor

@storopoli storopoli left a comment

Choose a reason for hiding this comment

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

ACK a577c22

@notmandatory notmandatory merged commit 66abc73 into bitcoindevkit:master May 8, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants