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

Test slither #2514

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Test slither #2514

wants to merge 5 commits into from

Conversation

jubeira
Copy link
Contributor

@jubeira jubeira commented Jun 6, 2023

Description

Slither test with v2-interfaces and v2-solidity-utils.

  • In interfaces, I'm getting 0 results after skipping everything in triage mode and re-running the analyzer, as expected.
  • In solidity-utils I still get (different) results when re-running the analyzer after skipping everything.

To run the test:

  • Clone the repository, and checkout the head of this branch
  • Run nvm use at repo root to select the right Node version
  • Run yarn at repo root
  • Run yarn slither-install at repo root
  • Change directory to pkg/solidity-utils and run yarn slither repeatedly. The output should be different between runs.

@jubeira jubeira added the do not merge Code playground or showcase - not intended to be merged label Jun 6, 2023
@jubeira
Copy link
Contributor Author

jubeira commented Jun 6, 2023

/cc @0xalpharush @montyly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Code playground or showcase - not intended to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants