Skip to content

BIP327,353: Update to Final and Proposed instead of Active #1980

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 24, 2025
Merged

BIP327,353: Update to Final and Proposed instead of Active #1980

merged 1 commit into from
Sep 24, 2025
+5 −5

Conversation

murchandamus
Copy link
Contributor

@murchandamus murchandamus commented Sep 23, 2025 •
edited by jonatack
Loading

A BIP status of Active is only for Process BIPs.

  • BIP 327 should be Final, not Active.
  • BIP 353 has an open PR with test vector additions, should be Proposed.

@murchandamus murchandamus mentioned this pull request Sep 23, 2025
Merged
jonatack
jonatack reviewed Sep 23, 2025
View reviewed changes
Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

Proposed might be appropriate, or wait a bit for Final, as 327 has #1964 and 353 has #1912.

@murchandamus murchandamus force-pushed the 2025-09-active-to-final branch from d2fc7a4 to a9d9dca Compare September 23, 2025 21:48
@murchandamus
BIP327,353: Correct statuses
cc68e6d 
- 327 should be Final instead of Active
- 353 should be Proposed, as Testvectors are still in the works (see
  bitcoin#1912)
@murchandamus murchandamus force-pushed the 2025-09-active-to-final branch from a9d9dca to cc68e6d Compare September 23, 2025 21:50
@murchandamus
Copy link
Contributor Author

murchandamus commented Sep 23, 2025 •
edited
Loading

I edited the OP.

My take is:

@jonatack jonatack changed the title BIP327,353: Update to Final instead of Active BIP327,353: Update to Final and Proposed instead of Active Sep 23, 2025
jonatack
jonatack approved these changes Sep 24, 2025
View reviewed changes
Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

ACK cc68e6d

@jonatack jonatack merged commit 898693d into bitcoin:master Sep 24, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonatack jonatack jonatack approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@murchandamus @jonatack