Closed Joelgranados closed 9 years ago
Looks pretty good to me. Will let chuong have a poke before we merge it though.
I am not sure how I can look at this. Just go ahead as you like.
Other than my line notes above, LGTM
You can rebase it after addressing those, or just merge it.
Both are related. The help needs to flow from the relative classes and the documentation is just a pretty way of presenting the help. its a big chunck of code, so I'll sprinkle some comments to ease the review.