Skip to content

chore(script): implement is_p2a for ScriptPubKeyExt - #5199

Merged
apoelstra merged 1 commit into
rust-bitcoin:masterfrom
luisschwab:chore/impl-is_p2a-on-ScriptPubKeyExt
Oct 27, 2025
Merged

chore(script): implement is_p2a for ScriptPubKeyExt#5199
apoelstra merged 1 commit into
rust-bitcoin:masterfrom
luisschwab:chore/impl-is_p2a-on-ScriptPubKeyExt

Conversation

@luisschwab

Copy link
Copy Markdown
Contributor

This PR implements the missing is_p2a() for the ScriptPubKeyExt trait.

Should probably be backported to 0.32.7.

@github-actions github-actions Bot added the C-bitcoin PRs modifying the bitcoin crate label Oct 26, 2025

@storopoli storopoli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ACK d41d8f2

@apoelstra

Copy link
Copy Markdown
Member

I'm about to ACK, but I realize that the mutant bot is going to complain about it. Can you add a unit test?

@apoelstra apoelstra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK d41d8f2; successfully ran local tests

@tcharding

Copy link
Copy Markdown
Member

We only currently mutation test primitives and units in CI so this one is ok as is. I had to dig so mentioning that its in .cargo/mutants.toml.

@tcharding tcharding left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK d41d8f2

@apoelstra
apoelstra merged commit be529cd into rust-bitcoin:master Oct 27, 2025
25 checks passed
@luisschwab
luisschwab deleted the chore/impl-is_p2a-on-ScriptPubKeyExt branch October 27, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-bitcoin PRs modifying the bitcoin crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants