Skip to content

feat(api): expose bitcoin types for the API#45

Merged
vladimirfomene merged 1 commit intobitcoindevkit:masterfrom
vincenzopalazzo:macros/api
May 9, 2023
Merged

feat(api): expose bitcoin types for the API#45
vladimirfomene merged 1 commit intobitcoindevkit:masterfrom
vincenzopalazzo:macros/api

Conversation

@vincenzopalazzo
Copy link
Copy Markdown
Contributor

In some cases, you need to use the bitcoin primitive to convert a type into another format or build a type from a hex string.

This commit will reexport the primitive used in the client without forcing the caller to add the bitcoin crate as direct dependency.

The motivation to have this is when you are interacting with a system that is not rust, and you do not need the rust-bitcoin.

In some cases, you need to use the bitcoin primitive to convert a
type into another format or build a type from a hex string.

This commit will reexport the primitive used in the client without
forcing the caller to add the bitcoin crate as direct dependency.

The motivation to have this is when you are interacting with a system
that is not rust, and you do not need the rust-bitcoin.

Signed-off-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
Copy link
Copy Markdown
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

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

ACK c52a9a0

Copy link
Copy Markdown
Contributor

@vladimirfomene vladimirfomene left a comment

Choose a reason for hiding this comment

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

Review ACK c52a9a0

@notmandatory notmandatory added this to the Release 0.5.0 milestone May 9, 2023
@notmandatory
Copy link
Copy Markdown
Member

@vladimirfomene once this one is merged I think it's time for a new release for this crate. Can we do it early next week?

@vladimirfomene
Copy link
Copy Markdown
Contributor

Yes, @notmandatory you read my mind.

@vladimirfomene vladimirfomene merged commit d571867 into bitcoindevkit:master May 9, 2023
@vincenzopalazzo vincenzopalazzo deleted the macros/api branch May 24, 2023 17:30
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 4591020650

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.077%

Totals Coverage Status
Change from base Build 4419789814: 0.0%
Covered Lines: 832
Relevant Lines: 1039

💛 - Coveralls

1 similar comment
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 4591020650

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.077%

Totals Coverage Status
Change from base Build 4419789814: 0.0%
Covered Lines: 832
Relevant Lines: 1039

💛 - Coveralls

chrono-raven6i90 added a commit to chrono-raven6i90/rust-esplora-client that referenced this pull request Oct 28, 2025
… types for the API

c52a9a08bc440300a74e311dde7c5f9ce0ea9d0f feat(api): expose bitcoin types for the API (Vincenzo Palazzo)

Pull request description:

  In some cases, you need to use the bitcoin primitive to convert a type into another format or build a type from a hex string.

  This commit will reexport the primitive used in the client without forcing the caller to add the bitcoin crate as direct dependency.

  The motivation to have this is when you are interacting with a system that is not rust, and you do not need the rust-bitcoin.

ACKs for top commit:
  notmandatory:
    ACK c52a9a08bc440300a74e311dde7c5f9ce0ea9d0f
  vladimirfomene:
    Review ACK c52a9a08bc440300a74e311dde7c5f9ce0ea9d0f

Tree-SHA512: 83a92dcfb4e8bbe6d6ca6ec2a420296cafd986122baafa252129c26d5fc176dd27ab21c40fb0b5d7d2106bb07e8964924607c2fc8f98602acae8b132730aedf9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants