-
Notifications
You must be signed in to change notification settings - Fork 979
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
Pausable printer includes checks on constructor() #1671
Comments
We need to add a check that like
slither/slither/core/declarations/function.py Line 389 in 64b2e0c
|
@aga7hokakological All yours! It'd be awesome if you submitted one PR each for this issue and #1670. Feel free to ask any questions here |
So the check is already there in I just moved this line in the above block and used continue keyword instead of False. It seems to work for |
Describe the issue:
Slither printer issues dialog that it wont report on constructors
Constructor and pure/view functions are not displayed
, then it reports on constructorsWhen fixing this, we should also consider the desired behavior if there are no applicable functions in a contract. Skip the contract all together, or list it with nothing beneath or
n/a
or somethingThis ticket could be completed along with #1670
The text was updated successfully, but these errors were encountered: