Skip to content

Commit

Permalink
github: remove benchmark step
Browse files Browse the repository at this point in the history
The --gas-report flag of the forge test command seems to conflict with
blobhash mocks in tests, making the new EIP-4844 tests fail, despite
they pass without this flag.

Turn off benchmarks temporarily, to unblock EIP-4844 development.
This should be debugged and fixed later.

Signed-off-by: Wojciech Zmuda <[email protected]>
  • Loading branch information
wzmuda committed Sep 11, 2024
1 parent 07e43ac commit 01b5472
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,9 @@ jobs:
run: make build

- name: Run Tests
run: make test
run: |
make test
make snapshot
- name: Check formatting
run: make format-check

- name: Run Benchmarks
run: |
make bench
make snapshot

0 comments on commit 01b5472

Please sign in to comment.