Remove boilerplate code. Remove check command.

adyatlov / bun-deprecated

DC/OS diagnostics bundle analysis tool

The Unlicense

12 stars 4 forks source link

Remove boilerplate code. Remove check command. #14

Closed adyatlov closed 6 years ago

adyatlov commented 6 years ago

I removed the check command to simplify usage. I added a CheckBuilder which dramatically simplified existing checks. I added the ReadFromJSON functions to simplify reading from JSON files. I got rid of some superficial things like contexts and progress reports. I abstracted the filesystem as the next step towards a test infrastructure. I documented some public types and functions.

adyatlov commented 6 years ago

As this pull request is quite big and I don't believe that at this stage someone would like to review it, I'm going ahead and merging it.