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

Make slither optional #1159

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Make slither optional #1159

merged 1 commit into from
Jan 8, 2024

Conversation

arcz
Copy link
Member

@arcz arcz commented Jan 8, 2024

Don't fail, instead print warning instead and continue. Includes a small refactor.

Closes #646 and #1137.

@arcz arcz requested a review from ggrieco-tob as a code owner January 8, 2024 23:25
@arcz arcz merged commit 9d502be into master Jan 8, 2024
18 checks passed
@arcz arcz deleted the optional-slither branch January 8, 2024 23:54
@aviggiano
Copy link

Hello

I was trying to disable slither entirely, even if it is installed on the machine. Is there a way to accomplish that with this current feature? If not, is it possible to include a --disable-slither type of flag that we can use?

@arcz
Copy link
Member Author

arcz commented Apr 16, 2024

There is no way to explicitly disable slither, can you open an issue for that?

@ggrieco-tob
Copy link
Member

@aviggiano I think you can create an empty bash script called slither in some directory (e.g. test) and then run echidna using PATH=./test:$PATH. That should "disable" slither.

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.

Add a flag to disable slither integration
3 participants