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

Removed unused features #11

Merged
merged 1 commit into from Mar 2, 2022
Merged

Removed unused features #11

merged 1 commit into from Mar 2, 2022

Conversation

Kixunil
Copy link
Collaborator

@Kixunil Kixunil commented Mar 2, 2022

After better analysis it was determined that these features don't help
with forward compatibility and only slightly improve convenience. The
noise is probably not worth it.

bitcoin_* were somewhat more interesting because the bitcoin crate
has sad no_std support but there's hope it'll be improved and even if
not adding features is better than removing. (The features weren't
released yet.)

Closes #10

After better analysis it was determined that these features don't help
with forward compatibility and only slightly improve convenience. The
noise is probably not worth it.

`bitcoin_*` were somewhat more interesting because the `bitcoin` crate
has sad `no_std` support but there's hope it'll be improved and even if
not adding features is better than removing. (The features weren't
released yet.)

Closes #10
@Kixunil Kixunil merged commit 0df26b0 into master Mar 2, 2022
@Kixunil Kixunil deleted the remove-unused-features branch March 2, 2022 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove unused features
1 participant