Skip to content

Create empty bitcoin-addresses crate#2928

Merged
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:06-27-bitcoin-addresses
Jul 1, 2024
Merged

Create empty bitcoin-addresses crate#2928
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:06-27-bitcoin-addresses

Conversation

@tcharding
Copy link
Copy Markdown
Member

@tcharding tcharding commented Jun 27, 2024

We intend on splitting the address types and logic out into a separate crate. In preparation for doing so, and so that we can grab the name on crates.io, add an empty crate bitcoin-addresses.

Tie it in to the CI infrastructure excluding the check-api infrastructure because that is in flux.

@tcharding
Copy link
Copy Markdown
Member Author

Alternative to #2898

@github-actions github-actions bot added test doc C-addresses PRs modifying the addresses crate labels Jun 27, 2024
@tcharding tcharding force-pushed the 06-27-bitcoin-addresses branch from f56ed59 to b0f795c Compare June 27, 2024 01:37
Kixunil
Kixunil previously approved these changes Jun 27, 2024
Copy link
Copy Markdown
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

ACK b0f795cb03d60b730d24315e87ecd8d7c6033c61

@apoelstra
Copy link
Copy Markdown
Member

Sorry, was procrastinating on this and now it needs rebase.

We intend on splitting the address types and logic out into a separate
crate. In preparation for doing so, and so that we can grab the name on
crates.io, add an empty crate `bitcoin-addresses`.

Tie it in to the CI infrastructure.
@tcharding tcharding force-pushed the 06-27-bitcoin-addresses branch from b0f795c to 0630457 Compare June 28, 2024 20:21
@tcharding tcharding closed this Jun 28, 2024
@tcharding tcharding deleted the 06-27-bitcoin-addresses branch June 28, 2024 20:22
@tcharding tcharding restored the 06-27-bitcoin-addresses branch June 28, 2024 20:37
@tcharding
Copy link
Copy Markdown
Member Author

Woops, wrong PR to close.

@tcharding tcharding reopened this Jun 28, 2024
Copy link
Copy Markdown
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

ACK 0630457

Copy link
Copy Markdown
Contributor

@junderw junderw left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 0630457 will try to tag and publish tonight, but may be tomorrow

@apoelstra apoelstra merged commit 2dd7963 into rust-bitcoin:master Jul 1, 2024
@apoelstra
Copy link
Copy Markdown
Member

Tagged and published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-addresses PRs modifying the addresses crate doc test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants