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.

Sign up for GitHub

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

Jump to bottom

Fail early if Soufflé requirement is not fulfilled #16

Closed
hiqua opened this issue Sep 26, 2018 · 2 comments
Closed

Fail early if Soufflé requirement is not fulfilled #16

hiqua opened this issue Sep 26, 2018 · 2 comments
Assignees
@hiqua
Labels
Enhancement New feature or request
Milestone
Sprint #1

Comments

@hiqua
Copy link
Contributor

hiqua commented Sep 26, 2018

Currently, Securify will throw a nullpointerexception if the requirement is not fulfilled, we should check whether the requirements are fulfilled from the beginning instead.
@hiqua hiqua added the Enhancement New feature or request label Sep 26, 2018
@ajhodges ajhodges mentioned this issue Oct 1, 2018
Closed
@MatthiasEgli-chainsecurity
Copy link
Collaborator

Didn't we implement this check already?

@MatthiasEgli-chainsecurity MatthiasEgli-chainsecurity added this to the Sprint #1 milestone Nov 5, 2018
@hiqua
Copy link
Contributor Author

hiqua commented Nov 5, 2018

Yes, if souffle is not available the error message is good enough.

@hiqua hiqua closed this as completed Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hiqua hiqua

Labels
Enhancement New feature or request
Projects
None yet
Milestone
Sprint #1
Development

No branches or pull requests
2 participants
@hiqua @MatthiasEgli-chainsecurity