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

Integrate Slither for continuous static analysis #214

Open
PaulRBerg opened this issue Dec 4, 2023 · 0 comments
Open

Integrate Slither for continuous static analysis #214

PaulRBerg opened this issue Dec 4, 2023 · 0 comments
Labels
effort: high Large or difficult task. priority: 3 Nice-to-have. Willing to ship without this. type: ci Changes to our CI configuration files and scripts. work: complicated Sense-analyze-respond. The relationship between cause and effect requires analysis or expertise.

Comments

@PaulRBerg
Copy link
Owner

PaulRBerg commented Dec 4, 2023

Slither v0.10.0 is compatible with PRBMath, so we can now configure it to run in CI like here:

https://github.com/PaulRBerg/prb-proxy/blob/245e2bd4ae9e3ac2eb632ef728f711e81ad6fbf5/.github/workflows/ci.yml#L152-L176

@PaulRBerg PaulRBerg added effort: high Large or difficult task. type: ci Changes to our CI configuration files and scripts. work: complicated Sense-analyze-respond. The relationship between cause and effect requires analysis or expertise. priority: 3 Nice-to-have. Willing to ship without this. and removed priority3 labels Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: high Large or difficult task. priority: 3 Nice-to-have. Willing to ship without this. type: ci Changes to our CI configuration files and scripts. work: complicated Sense-analyze-respond. The relationship between cause and effect requires analysis or expertise.
Projects
None yet
Development

No branches or pull requests

1 participant