* ci: add slither

* ci: slither code scanning integration

* ci: change configs

* build: bump pragma

* build: change pragmas

* docs: add WIP note

* ci: add scopelint to ci

* fix: add foundry install step to CI

* style: move fmt before slither so both slither tasks are adjacent in PRs

* wrong solidity formatting to test ci fmt job

* fix solidity and break toml formatting to test fmt ci job

* restore formatting

* one test should error

* chore: fix naming convention + exclude that slither check

* add unused function to fail slither and update ci

* dummy commit

* add comment

* updates

* chore: put ci profile first since it inherits default solc, fix typo

* docs: update README

* build: update forge-std

* ci: download scopelint binary

* ci: fix repo

* ci: try different binary

* docs: update README

* ci: remove 'install deps' step, foundry does this automatically

* ci: initial coverage check

* chore: install deps for coverage (not yet auto installed)

* ci: install lcov

* ci: forgot to generate lcov

* ci: comment out the lcov directory filter

* ci: filter out test and script dirs by default

* refactor: use deploy script to scaffold tests, follow best practice test structure + naming conventions

* docs: update docs

* docs: add info on coverage

* ci: forge coverage now installs missing deps

* ci: log scopelint version

* ci: remove outdated rust toolchain install

* chore: comment out broadcast call in sample script for now

* chore: add lcov.info to gitignore

* ci: preserve coverage branch info

* docs: update best practices link and coverage section

* ci: also print coverage summary in CI (though it won't be filtered)

* doc: add comments

* ci: add second coverage tool

* try another

* maybe remove versioning fixes other tool

* fixes

* test comment deletion

* test title

* remove title

* test

* always run

* restore normal counter file

* cleanup ci file

docs: update note about caching

* ci: fix bug in "slither-analyze" job in ci.yml

* ci: change the target from "contracts/" to "src/"

* ci: add "security-events" write permission to "slither-analyze" job in ci.yml

* ci: remove redundant "filter-paths" arg from "slither-analyze" job

…ift#16)

* ide gitignore

* style: capitalization

---------

Co-authored-by: Matt Solomon <[email protected]>

* chore: disable slither reentrancy detector in scripts

* build: pin to 0.8.20

* chore: absolute imports

* ci: ignore solc-version check

this check is subjective, and I don't think project should necessarily
follow the version recommendations from slither (partly because historically
the versions suggested by that check were not updated frequently)

* build: explicitly specify evm version to protect against foundry changing the default

* docs: update README