ci: add slither static analysis checks
Motivation
Describe why this issue should be fixed and link to any relevant design docs, issues or other relevant items.
Change Summary
Describe the changes being made in 1-2 concise sentences.
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review
- [ ] The PR title adheres to the conventional commits standard
- [ ] The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
- [ ] The PR's changes adhere to all the requirements in the contribution guidelines
- [ ] This PR does not require changes to the Farcaster protocol
Additional Context
If this is a relatively large or complex change, provide more details here that will help reviewers.
Codecov Report
Patch and project coverage have no change.
Comparison is base (
8ae8be1) 84.78% compared to head (507fd39) 84.78%.
Additional details and impacted files
@@ Coverage Diff @@
## main #197 +/- ##
=======================================
Coverage 84.78% 84.78%
=======================================
Files 8 8
Lines 309 309
Branches 81 81
=======================================
Hits 262 262
Misses 45 45
Partials 2 2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Coverage after merging varunsrin/add-slither-part-2 into main will be
| 94.23% |
|---|
closing for now, don't think we want slither in ci just yet