AST should include import information and ...

Consensys / armlet

a MythX API client wrapper

MIT License

16 stars 7 forks source link

AST should include import information and ... #87

Closed rocky closed 5 years ago

rocky commented 5 years ago

add --yaml option
mention sabre-mythx

@b-mueller the changes here to the compile command to include the AST are relevant to sabre-mythx as well.

Also we should come up with a strategy for what goes wehre between this code and the other. My understanding is that sabre-mythx was to a Minimal Viable Product (MVP) of less than 100 lines. Is that still the case?

mythx-analysis started out as an example program as well.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 386

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.465%

Totals
Change from base Build 385:	0.0%
Covered Lines:	247
Relevant Lines:	299

💛 - Coveralls

rocky commented 5 years ago

We can come back after release to address any problems.