Skip to content

Use slither 0.8.3 in CI - #96

Closed
nkuba wants to merge 2 commits into
mainfrom
slither-0.8.3
Closed

nkuba wants to merge 2 commits into
mainfrom
slither-0.8.3

Conversation

@nkuba

@nkuba nkuba commented Apr 26, 2022

Copy link
Copy Markdown
Member

Use the latest version of slither 0.8.3

Use the latest version of slither 0.8.3
I upgraded dependencies to OpenZeppelin's packages as this was suggested
in one of the slither's issues as possible fix to slither's crash:
crytic/slither#1140
@nkuba nkuba changed the title Use slither 0.8.2 in CI Use slither 0.8.3 in CI Apr 26, 2022
@nkuba

nkuba commented Apr 26, 2022

Copy link
Copy Markdown
Member Author

Blocked by crytic/slither#1140

@piotr-roslaniec
piotr-roslaniec dismissed stale reviews from ghost September 2, 2025 14:09

Dismissing fraudulent review from compromised account (Incident: THRESH-SEC-2025-001)

@mswilkison

Copy link
Copy Markdown
Contributor

Superseded by #187, which tracks upgrading Slither from the current CI pin of 0.8.0 to a current stable release (0.11.6 as of 2026-09-08).

The replacement task coordinates with the toolchain refresh in #182 and requires rechecking the original OpenZeppelin-related crash, reviewing detector changes, and demonstrating passing CI. Closing this stale 0.8.3 draft; the upgrade work is tracked in #187.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants