Foundry changed the test command output, changing the "Test result" to "Suite result", causing the command gas-report to stop working. See https://github.com/foundry-rs/foundry/pull/722817215eb2f7
Updating the word on the gas-report command fixes the problem.
This introduces a new `pnpm release` command in our template which makes
use of `commit-and-tag-version`, that will generate changelogs based on
our project's commit histories.
This adds a new command to package.json `verify` which can be run via
`pnpm verify`.
The command runs the certora CLI with a config file which has to be
adjusted for every individual project.
The commit also adds a dedicated task to our github actions, which
ensures, verification is done in every PR as well.
To allow deployment to different chains but using the same deployment
scripts, this commit introduces a basic `DeploymentConfig` which can be
extended as necessary in each project.
There's a few things that should be considered:
- `activeNetworkConfig` will be initialized via the constructor, at
which point it is know what `block.chainid` is
- To add new configuration settings, extend `NetworkConfig`
- To add a new config for a different chain, extend the `if/else` block
in the constructor so that it creates a `NetworkConfig` for the chain
in question
This is an initial take on adjusting the foundry-template to the Vac's
smart contract unit's needs. In a nutshell what this does is:
1. Update README to not refer to upstream repo where not necessary
2. Removes PRBergTest library from dependencies
3. Removes `FUNDING.qml`
4. Adjust CI actions
There are more things to be done and decided on in follow-up commits.
* use solhint-community fork
We're trying to keep solhint afloat with a fork, and done some work on
it so far, mainly the no-unused-imports rule.
Any feedback is appreciated, hope this helps ✨
see announcement:
https://blog.capu.tech/announcing-solhint-community.html
* docs: update README
---------
Co-authored-by: Paul Razvan Berg <paul.razvan.berg@gmail.com>