Skip to content

Commit

Permalink
👷 enable --ir-minimum flag for forge coverage
Browse files Browse the repository at this point in the history
The coverage commands do not run the optimizer, meaning it is more
sensible to the stack depth error. We have reached a point where our stack
management can't do without the optimizer, that's why we have to enable
this flag for the coverage command.
  • Loading branch information
qd-qd committed Dec 15, 2023
1 parent 1ec5bcc commit 32be9fb
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -182,9 +182,9 @@ jobs:

- name: "Generate the coverage report"
# contracts in the test/ and script/ directory are excluded fron the report
# the precompute internal version of the library is also excluded from the report as
# it is highly experimental and to meant to be used at all
run: "forge coverage --report lcov && lcov --remove lcov.info \
# the flag `--ir-minimum` is required as the coverage report is generated without the optimization steps
# please remove it as soon as it is no longer needed
run: "forge coverage --ir-minimum --report lcov && lcov --remove lcov.info \
-o lcov.info 'test/*' 'script/*'"

- name: "Add coverage summary"
Expand Down

0 comments on commit 32be9fb

Please sign in to comment.