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

Chip13 (#15688) #15825

Merged
merged 3 commits into from
Jul 21, 2023
Merged

Chip13 (#15688) #15825

merged 3 commits into from
Jul 21, 2023

Conversation

wallentx
Copy link
Contributor

  • Chip13

  • Lint.

  • Black.

  • Lint.

  • Fix BlockTools.

  • Trigger the soft fork.

  • Parametrize bt.

  • Syntax.

  • Add softfork3 to empty_blockchain too.

  • Skip test.

  • Skip test.

  • Try fix.

  • Add persistent blocks.

  • Lint.

  • Run tests without block cache.

  • Try to fix tests.

  • Revert "Try to fix tests."

This reverts commit f10c7fc.

  • WIP plot filters.

  • pytest.

  • Lint.

  • Skip cache tests.

  • Try to lower the constraints.

  • Try to increase timeout.

  • Test blockchain.

  • Store test.

  • Revert to main

  • Revert to main

  • Revert to main

  • Add tests with new interface

  • Test.

  • Fixtures.

  • Fixtures.

  • Lint.

  • Add first test.

  • Improve test.

  • Isort.

  • merge fix

  • fix counter

  • Bump test-cache version to soft_fork3.0 DBs

  • Enable tests.

  • Enable all.

  • Syntax.

  • Attempt higher timeouts.

  • Try to skip big test.

  • Revert tests larger timeouts.

  • Try to get test to work with 4000.

  • Skip 10000 blocks.

  • Test.

  • Update test_full_node_store.py

  • Update test_blockchain.py

  • Update test_blockchain.py

  • Update tests/conftest.py

  • Update tests/blockchain/test_blockchain.py

  • Address some comments.

  • Some more comments.

  • Use new passes_plot_filter signature.

  • Lint.

  • Lint.

  • Reduce benchmark timeout.

  • Skip test.

  • Address review comments.

  • Delete leftovers.

  • Review comments.

  • await.

  • Revert async call.

  • Validate in body too.

  • Fix test.


Purpose:

Current Behavior:

New Behavior:

Testing Notes:

* Chip13

* Lint.

* Black.

* Lint.

* Fix BlockTools.

* Trigger the soft fork.

* Parametrize bt.

* Syntax.

* Add softfork3 to empty_blockchain too.

* Skip test.

* Skip test.

* Try fix.

* Add persistent blocks.

* Lint.

* Run tests without block cache.

* Try to fix tests.

* Revert "Try to fix tests."

This reverts commit f10c7fc.

* WIP plot filters.

* pytest.

* Lint.

* Skip cache tests.

* Try to lower the constraints.

* Try to increase timeout.

* Test blockchain.

* Store test.

* Revert to main

* Revert to main

* Revert to main

* Add tests with new interface

* Test.

* Fixtures.

* Fixtures.

* Lint.

* Add first test.

* Improve test.

* Isort.

* merge fix

* fix counter

* Bump test-cache version to soft_fork3.0 DBs

* Enable tests.

* Enable all.

* Syntax.

* Attempt higher timeouts.

* Try to skip big test.

* Revert tests larger timeouts.

* Try to get test to work with 4000.

* Skip 10000 blocks.

* Test.

* Update test_full_node_store.py

* Update test_blockchain.py

* Update test_blockchain.py

* Update tests/conftest.py

Co-authored-by: Arvid Norberg <arvid@libtorrent.org>

* Update tests/blockchain/test_blockchain.py

Co-authored-by: Arvid Norberg <arvid@libtorrent.org>

* Address some comments.

* Some more comments.

* Use new passes_plot_filter signature.

* Lint.

* Lint.

* Reduce benchmark timeout.

* Skip test.

* Address review comments.

* Delete leftovers.

* Review comments.

* await.

* Revert async call.

* Validate in body too.

* Fix test.

---------

Co-authored-by: almog <almogdepaz@gmail.com>
Co-authored-by: wallentx <william.allentx@gmail.com>
Co-authored-by: Arvid Norberg <arvid@libtorrent.org>
@wallentx wallentx requested a review from a team as a code owner July 21, 2023 03:59
@wallentx wallentx added the Added Required label for PR that categorizes merge commit message as "Added" for changelog label Jul 21, 2023
wjblanke
wjblanke previously approved these changes Jul 21, 2023
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

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

aok

Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

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

aok

@wallentx wallentx merged commit 97245c9 into release/2.0.0 Jul 21, 2023
480 of 481 checks passed
@wallentx wallentx deleted the wallentx/cherrychip branch July 21, 2023 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants