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

Add gas tracking features #75

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add gas tracking features #75

wants to merge 5 commits into from

Conversation

gvladika
Copy link
Contributor

@gvladika gvladika commented Feb 9, 2024

Adds gas tracking actions

yarn run gas:snapshot generates the referent snapshot
CI runs yarn run gas:check to test there is any diffs compared to referent values
yarn run gas:report prints out a gas report for contract function

Add a special version of gas reporter which prints out the gas diff per function call, for current implementation of specific code vs. the deployed version of the code. This reported can be used like this:

❯ yarn run gas:report:benchmark

❯ yarn run gas:report:benchmark
$ ts-node test-gas-reporter/gasReportBenchmark.ts
Gas diff compared to referent report:
outboundTransfer: -1445 (-0.82%)

@cla-bot cla-bot bot added the cla-signed label Feb 9, 2024
@gzeoneth
Copy link
Member

I found the gas snapshot ci pretty annoying with the AF governance repo I am not sure we want it here, idk

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