Readme update - Githubissues

Consensys / armlet

a MythX API client wrapper

MIT License

17 stars 7 forks source link

Readme update #38

Closed rocky closed 5 years ago

rocky commented 5 years ago

Thoughts?

rocky commented 5 years ago

I should say as an aside that I had wanted to allow the clientToolName to be set as a default in the constructor which could then be overridden in the analyze call, but that broke the test, and I was exhausted and so I didn't want to deal with that.

I mention someone else thinks this is a good idea and wants to run with it.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 192

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 88.35%

Totals
Change from base Build 184:	0.0%
Covered Lines:	192
Relevant Lines:	217

Consensys / armlet

Readme update #38

Pull Request Test Coverage Report for Build 192

💛 - Coveralls