Skip to content

Conversation

notmandatory
Copy link
Member

@notmandatory notmandatory commented Nov 11, 2023

Description

As suggested by @TheBlueMatt we shouldn't use the log crate because even though it is a rust-lang project is may depend on some other crates that aren't as well maintained. Since we were only use log in a few places and not it any other bdk workspace projects except the bdk package I removed it.

Notes to the reviewers

For consistency I also removed the env_logger from the dev dependencies since it was only being used in a few places in a couple examples and println! is adequate for examples.

Changelog notice

None

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 this to the 1.0.0-alpha.3 milestone Nov 11, 2023
@notmandatory notmandatory self-assigned this Nov 11, 2023
Copy link
Member

@evanlinjin evanlinjin left a comment

Choose a reason for hiding this comment

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

ACK 18e9e0a

@danielabrozzoni
Copy link
Member

Should we open an issue to introduce some sort of diy logger as ldk does?

ACK 18e9e0a

@notmandatory notmandatory mentioned this pull request Nov 16, 2023
@notmandatory
Copy link
Member Author

Should we open an issue to introduce some sort of diy logger as ldk does?

I just created a placeholder issue.

@notmandatory
Copy link
Member Author

rebased

@notmandatory notmandatory merged commit 73862be into bitcoindevkit:master Nov 20, 2023
@notmandatory notmandatory deleted the chore/remove_log_dep branch May 26, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants