Skip to content

Comments

Fix up black and mypy in fix-t8n-requests#2

Merged
petertdavies merged 2 commits intopetertdavies:fix-t8n-requestsfrom
danceratopz:fix-t8n-requests-tox
Nov 25, 2024
Merged

Fix up black and mypy in fix-t8n-requests#2
petertdavies merged 2 commits intopetertdavies:fix-t8n-requestsfrom
danceratopz:fix-t8n-requests-tox

Conversation

@danceratopz
Copy link

@danceratopz danceratopz commented Nov 25, 2024

What was wrong?

Fixes tox -e static, which is failing in ethereum#1040.

Couldn't suggest the mypy fix in a review, so adding here.

Cute Animal Picture

Cute hedgehog

@petertdavies petertdavies merged commit 5e35d1d into petertdavies:fix-t8n-requests Nov 25, 2024
petertdavies pushed a commit that referenced this pull request Jan 31, 2025
* dev: change type of branch node subnodes

* Annotate tuple with 16

* add ellipsis to tuple

* explicit tuple

* use explicit 16-field tuple

* fmt

* remove Annotated import

* fix types

* Use assert_type alongside cast (#1)

* Import from typing_extensions instead of typing (#2)

---------

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
petertdavies pushed a commit that referenced this pull request Jan 22, 2026
* fix(t8n): introduce profiler

* profiler pauses

* fix(testing/base_types): Speed up to_bytes

fix(base_types): unnecessary cast

* fix(testing/base_types): CoerceBytes

* fix(testing): Optimize pre-alloc grouping state hash

* feat(t8n): introduce LazyAlloc

* feat(fixtures): Cached genesis model dump

* refactor: prefer explicit cache over circular reference (#2)

* Update packages/testing/src/execution_testing/client_clis/cli_types.py

Co-authored-by: spencer <spencer.tb@ethereum.org>

* fix: verify modified gas limit

* refactor: Use generics

* refactor: Add state root cache

* docs: changelog

* feat(client_clis): Add unit tests

---------

Co-authored-by: danceratopz <danceratopz@gmail.com>
Co-authored-by: spencer <spencer.tb@ethereum.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants