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

remove chip13 #15871

Merged
merged 6 commits into from
Jul 27, 2023
Merged

remove chip13 #15871

merged 6 commits into from
Jul 27, 2023

Conversation

wjblanke
Copy link
Contributor

Purpose:

Remove CHIP13 from release

Current Behavior:

CHIP13 at soft fork3

New Behavior:

No CHIP13 at soft fork3

Testing Notes:

n/a

@wjblanke wjblanke added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Jul 26, 2023
@wjblanke wjblanke requested a review from a team as a code owner July 26, 2023 20:49
@wjblanke wjblanke requested review from arvidn and emlowe July 26, 2023 20:51
@emlowe
Copy link
Contributor

emlowe commented Jul 26, 2023

Some flake8 failures due to unused imports...

@arvidn
Copy link
Contributor

arvidn commented Jul 26, 2023

@wjblanke I think we need a note to make sure this PR is not check-pointed back into main later

arvidn
arvidn previously approved these changes Jul 26, 2023
@arvidn arvidn requested a review from fchirica July 26, 2023 22:02
@arvidn
Copy link
Contributor

arvidn commented Jul 26, 2023

@wallentx wallentx merged commit 4013451 into release/2.0.0 Jul 27, 2023
@wallentx wallentx deleted the nochip13wjb branch July 27, 2023 02:00
wjblanke added a commit that referenced this pull request Aug 3, 2023
* remove chip13

* flake8

* flake8

* flake8

* isort

* Remove chip-13 test

---------

Co-authored-by: Earle Lowe <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants