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.

Sign up for GitHub

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

Jump to bottom

Switch from Solhint to Slither #187

Closed

CodeSandwich opened this issue Aug 11, 2022 · 1 comment · Fixed by #266

Collaborator

CodeSandwich commented Aug 11, 2022

solhint isn't a very good linter anymore:

It's not very helpful. It very rarely finds anything and I can't recall a single time when its warning actually improved the code quality, these were always annoying nitpicks which forced flagging code to be ignored.
It's basically abandoned. The repository hasn't been pushed to in over a year and minimalistic Solidity-bumping updates are released by a single person without access to the upstream.

Slither seems to be an alternative worth trying out. It's actively developed and seems to be widely praised among developers.

The text was updated successfully, but these errors were encountered:

Collaborator Author

CodeSandwich commented Aug 17, 2022 •

edited

Loading

Blocked by crytic/slither#1352 (being fixed in crytic/slither#1378), unless it's implemented, Slither can't analyze this codebase.

CodeSandwich mentioned this issue

Move tests from ./src/test to ./test #230

Merged

CodeSandwich mentioned this issue

Switch to Slither #266

Merged

CodeSandwich linked a pull request

that will close this issue

Switch to Slither #266

Merged

CodeSandwich closed this as completed in #266

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