Skip to content

Commit

Permalink
ci: solidity gas diff options [SLT-267] (#3193)
Browse files Browse the repository at this point in the history
* ci: ignore test files in gas diff report

* add some changes to the test files

* ci: define some options for gas-diff

* try changing the contract to trigger gas diffs

* Revert "try changing the contract to trigger gas diffs"

This reverts commit 4504e3c.

* Revert "add some changes to the test files"

This reverts commit 7e7d6cb.
  • Loading branch information
ChiTimesChi authored Sep 26, 2024
1 parent 1bdfee6 commit 2709e85
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/solidity.yml
Original file line number Diff line number Diff line change
Expand Up @@ -276,9 +276,12 @@ jobs:

- name: Compare gas reports
uses: Rubilmax/foundry-gas-diff@v3.18
# TODO: define some options?
with:
ignore: 'test/**/*'
report: 'gas-report-${{ matrix.package }}.ansi'
sortCriteria: avg
sortOrders: desc
summaryQuantile: 0.5
id: gas_diff

- name: Add gas diff to sticky comment
Expand Down

0 comments on commit 2709e85

Please sign in to comment.