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

Update slither config #828

Merged
merged 2 commits into from
Nov 30, 2023
Merged

Update slither config #828

merged 2 commits into from
Nov 30, 2023

Conversation

zajck
Copy link
Member

@zajck zajck commented Nov 30, 2023

Change slither config to:

  • Exclude contract examples, since they are not part of the protocol
  • Ignore solc-version detector, since we often use a newer version than recommended by slither

@zajck zajck changed the title exclude examples and solc-version detector Update slither config Nov 30, 2023
@coveralls
Copy link

coveralls commented Nov 30, 2023

Coverage Status

coverage: 98.639%. remained the same
when pulling 2f97c8d on slither-config
into 25fa3cf on main.

@zajck zajck self-assigned this Nov 30, 2023
@zajck zajck merged commit fe19871 into main Nov 30, 2023
12 checks passed
@zajck zajck deleted the slither-config branch November 30, 2023 11:42
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.

4 participants