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

Fix indentation for some kzg_7594 tests #3855

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

jtraglia
Copy link
Member

Somehow (probably because of a merge conflict) the last three verify_cell_kzg_proof_batch test cases were indented too far and as a result were part of the loop above, so they were being executed multiple times. This doesn't change the generated tests because it would try to do the same test with the same inputs. Just want to fix these "skipping" messages.

image

Somehow (probably because of a merge conflict) the last three
verify_cell_kzg_proof_batch test cases were indented too far and as a result
were part of the loop above, and as a result executed multiple times. This
doesn't change the generated tests because it would try to do the same test
with the same inputs.
@jtraglia jtraglia added the EIP-7594 PeerDAS label Jul 25, 2024
@hwwhww hwwhww merged commit e9be4e2 into ethereum:dev Jul 25, 2024
26 checks passed
@jtraglia jtraglia deleted the kzg-7594-tests-indentation branch July 25, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants