Skip to content

BIP 370: drop redundant requirement from PSBT_OUT_SCRIPT field description#2111

Open
darosior wants to merge 1 commit intobitcoin:masterfrom
darosior:2603_bip370_output_amount
Open

BIP 370: drop redundant requirement from PSBT_OUT_SCRIPT field description#2111
darosior wants to merge 1 commit intobitcoin:masterfrom
darosior:2603_bip370_output_amount

Conversation

@darosior
Copy link
Member

@darosior darosior commented Mar 3, 2026

As per #2111 (comment):

The requirement was already stated by the "Versions Requiring Exclusion" and "Versions Requiring Inclusion" columns.

@darosior
Copy link
Member Author

darosior commented Mar 3, 2026

cc @achow101

@achow101
Copy link
Member

achow101 commented Mar 3, 2026

The requirement was already stated by the "Versions Requiring Exclusion" and "Versions Requiring Inclusion" columns.

It would be more consistent with the rest of the fields to remove that sentence from PSBT_OUT_SCRIPT.

@jonatack jonatack added Proposed BIP modification Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified labels Mar 4, 2026
…iption

There are already "Requiring inclusion" / "Requiring exclusion" columns that specify that.
@darosior darosior force-pushed the 2603_bip370_output_amount branch from 03b931d to 175790e Compare March 4, 2026 15:57
@darosior darosior changed the title BIP 370: add missing requirement for PSBT_OUT_AMOUNT field BIP 370: drop redundant requirement from PSBT_OUT_SCRIPT field description Mar 4, 2026
@darosior
Copy link
Member Author

darosior commented Mar 4, 2026

Right, updated the commit to do just that.

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

Labels

Pending acceptance This BIP modification requires sign-off by the champion of the BIP being modified Proposed BIP modification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants