Skip to content
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.

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

Softfork2 infrastructure #14625

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Softfork2 infrastructure #14625

merged 2 commits into from
Feb 21, 2023

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Feb 20, 2023

introduce the new constant for the new upcoming soft-fork (for 1.8.0). Also enable parsing the ASSERT_BEFORE_* conditions after that height.

This flag also enables the other new conditions like ASSERT_CONCURRENT_SPEND and ASSERT_CONCURRENT_PUZZLE.

Note that these are already enabled in MEMPOOL_MODE unconditionally.

@arvidn arvidn added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Feb 20, 2023
@arvidn arvidn force-pushed the softfork2-infrastructure branch from 2ba60b7 to 65b64a7 Compare February 20, 2023 14:13
@arvidn arvidn marked this pull request as ready for review February 20, 2023 14:45
@arvidn arvidn requested a review from a team as a code owner February 20, 2023 14:45
@richardkiss
Copy link
Contributor

This seems simple enough.

@richardkiss richardkiss self-requested a review February 21, 2023 02:19
@arvidn arvidn added the ready_to_merge Submitter and reviewers think this is ready label Feb 21, 2023
@wallentx wallentx merged commit 1d4a3ec into main Feb 21, 2023
@wallentx wallentx deleted the softfork2-infrastructure branch February 21, 2023 18:25
emlowe pushed a commit that referenced this pull request Apr 11, 2023
* add soft-fork2 (to be introduced in 1.8.0)

* update unknown-opcode test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants