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

Hack up a -jra version of this crate #171

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Mar 24, 2025

Just Release Already.

If you do not have a PR up within two weeks @Kixunil that puts this crate in a releasable state then I am forking and releasing hex-conservative-jra 1.0 with what is in this PR.

If we then still cannot come to an agreement I will fork all the way up to rust-bitcoin. I'm done arguing with you bro, we need to just fucking release already.

You have two weeks.

Mention the new `decode_` functions in the crate level docs.
IMO this crate is close to ready for release. Introduce an API script to
help verify the exact API.
Run the recently introduced script with:

  contrib/check-for-api-changes.sh

and commit the files. No manual changes.
We made a bunch of changes to the error types in an effort to make it
possible to update them without breaking semver - this is a lost cause.

Simplify the error types and just fucking commit to the API.

While we are at it make the `error` module private because it is a code
organization thing only. Re-export all the error types at the crate root.
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.

1 participant