-
Notifications
You must be signed in to change notification settings - Fork 982
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
Add the support for multiple compilation units. #823
Conversation
lot of breaking changes: - SlitherCore is split into SlitherCore and SlitherCompilationUnit, as a result several functions from Slither are not direclty accessible anymore - There is a list of compilation unit in SlitherCore - The detectors are run per compilation unit - however slither filters out results if their ID are the same to avoid dupplicate - The printers are run per SlitherCore. - The non-crytic-compile inputs are removed (json/list of json). This will help us to clean the API in the long run. These were only used in internal tests - slither.get_contract_from_name returns now a list (which can be empty)
This pull request fixes 1 alert when merging 73764ed into cf07c59 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 71e33c6 into cf07c59 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 84f7605 into cf07c59 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 6233f31 into cf07c59 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 9533947 into 4a8cdf7 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging ceb7478 into 4a8cdf7 - view on LGTM.com fixed alerts:
|
This pull request fixes 1 alert when merging 3f3855a into 4a8cdf7 - view on LGTM.com fixed alerts:
|
This PR includes a lot of breaking changes:
anymore
There are a couple of open questions left. For it's difficult to know what a compilation unit refers too, so for the tool that ask for a contract name (ex:
slither-flat
), and the contract is shared by multiple compilation unit, it is unclear what to do