Closed Roflicide closed 7 years ago
@zanix I'm not sure if I incorporated all of your changes from the last pull request into this one; I apologize if I haven't--please update it as you see fit
@Roflicide ok, I will try to take a look this weekend.
This looks good. Thanks. I want to take a closer look at it again later this weekend though.
I think you got all my comments from the other PR. Hopefully I can review this PR entirely tomorrow.
I had a chance to look over this again, and I like the changes. I'll wait for @Zanix to give comments before accepting this change. It would be nice to get my pull request (#166) in first since the wording here assumes that it is already in.
Finally had a chance to review, I think it looks good. @sandain I agree with getting that other PR in first.
Updated the docs to include instructions on setting up Overviewer; this includes much of the same README language as pull request #160 except: