ethereum / ethereum/execution-specs

`accessList` or `accessLists`

Open
#1,194 3 comments 2 reactions 1 assignee Claimed by @gurukamath View on GitHub
A-spec-tools E-easy P-medium stale
Dominant language
Python
Stars
1.2k
Forks
505
Avg merge
2d 14h
Merged PRs (30d)
116

Description

> There seems to be a problematic commit in `forks/prague`: [8f89be5](https://github.com/ethereum/execution-specs/commit/8f89be57c9f1c764d1fa5b95eb32db0eac7fd3d6)
>
> If I revert this commit, `7702-to-precompile` fills all tests on EEST main a1c4eeff34.

The problem here seems to be the fact that all other `t8n` tools expect the variable `accessList` and eels-t8n expects `accessLists`. The state test fixtures also, provide `accessLists`.

So in the long run it might be good to align on one variable name. But in the short term, in the interest of avoiding changes to 5 different client t8n tools, eels-t8n can also start expecting `accessList`. I will push a commit with the hot fix in this PR and create an issue to track this in the long term.

_Originally posted by @gurukamath in https://github.com/ethereum/execution-specs/issues/1191#issuecomment-2804261344_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.