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

Improve the human summary printer #228

Merged
merged 8 commits into from May 7, 2019

Conversation

Projects
None yet
2 participants
@montyly
Copy link
Member

commented May 7, 2019

Several of the new features are integrated within slither.core or slither.utils

  • Add state_variable.signature function(s) to translate a state variable to a function signature
  • Add several contract.is_erc_* methods
  • Add contract.is_from_dependency that checks if the contract is from a dependency
  • Add utils.erc with standard ERCs signatuers
  • Add utils.standard_libraries with standard libraries

Several of these methods require crytic_compile 3ed5160924d08a4d3c86d7cddf9f1e17dfe51808

Close #220

Xenomega and others added some commits Apr 26, 2019

-Relaxed erc20-interface detector to report incorrect function signat…
…ures even if the function was not declared in that contract immediately

-Created erc721-interface detector, similar to erc20-interface.
Improvements of the human summary printer.
Several of the new features are integrated within slither.core or slither.utils

- Add state_variable.signature function(s) to translate a state variable to a function signature
- Add several contract.is_erc_* methods
- Add contract.is_from_dependency that checks if the contract is from a dependency
- Add utils.erc with standard ERCs signatuers
- Add utils.standard_libraries with standard libraries

Several of these methods require crytic_compile 3ed5160924d08a4d3c86d7cddf9f1e17dfe51808

@montyly montyly changed the base branch from master to dev May 7, 2019

@montyly montyly merged commit f8244eb into dev May 7, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details

@montyly montyly deleted the dev-human-summary-printer branch May 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.