-
Notifications
You must be signed in to change notification settings - Fork 4
Fit to template repo #48
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
Conversation
Hardhat + ethers v6 + foundry
* prettier * eslint running * package json format and lint commands * tsconfig options * update readme * Formatting ci (#4) * use node 18
* prettier * eslint running * package json format and lint commands * tsconfig options * update readme * Formatting ci (#4) * use node 18 * Test ci * forge install * test contract size * test gas check * add snapshot * readme * fix formatting * upload all selectors * test-sigs * test-sigs ci * build before test-sigs * test-storage * readme * readme * unit folder * unit folder
* fork tests * fork tests * unusded imports * fork tests ci * fmt * gas snapshot * match path in gas check * ... * bash scripts for unit and fork tests * globstar * globstar in package json * multichain fork tests * change exit code * spacing * update fork test job * secrets * secrets * secrets * ...
* e2e test * fmt * comment * compile * ignore lib * localhost to 127 * forge install for lint * setup-sdk * compile before lint * compile in e2e script * ... * prepare * ... * ethersv5 * install foundry in install-node job * reinstall node modules every time * ... * readme * ... * custom fee l3 * always orbit, test both eth and custom fee * format * name job * readme * testSetup * fmt * ...
* e2e test * fmt * comment * compile * ignore lib * localhost to 127 * forge install for lint * setup-sdk * compile before lint * compile in e2e script * ... * prepare * ... * ethersv5 * install foundry in install-node job * reinstall node modules every time * ... * readme * ... * custom fee l3 * always orbit, test both eth and custom fee * format * name job * readme * testSetup * fmt * ... * audit * readme * ci job * ... * readme
* minimal-publish * fmt * readme * ...
* separate files for jobs * ... * use v4 * update readme * update readme
* chore: yarn fix * ci: fix yarn install
* disable script * update readme and jobs * update test-e2e.bash * update readme * ...
fix: script path
minimal-install
* minimal-install * lighter e2e * update readme and comment out l3 jobs
This reverts commit 7d59c9f.
# uses: actions/setup-node@v4 | ||
# with: | ||
# node-version: 18 | ||
# node-version: 22 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we wanna keep these commented out workflows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah i think we should just to keep the diff with the template smaller
solc_version = '0.8.16' | ||
optimizer = true | ||
optimizer_runs = 20000 | ||
via_ir = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is via_ir false the default? do we want to keep it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah ir is default disabled
No description provided.