Power to the author
More features for PR and comment authors
Reviewable just got a few new features that relax some restrictions on the review process. First, it used to be that once you published a comment, the only way to resolve it (and take it off the “progress counters”) was to get somebody else to acknowledge it. Even if you changed your mind and decided it was no longer relevant. Now you can resolve your own comment, as long as nobody else has replied or acknowledged it yet:
The restriction on a pull request author marking files as reviewed is now also relaxed. While you'd normally want other people to review your code, there are some situations where it makes sense for the PR author to review things as well: perhaps some of the PR's commits have been contributed by other developers, or you're working solo and using Reviewable to review your own code. The mark as reviewed buttons have a slightly different look in that case but work just the same. To increase transparency (and help keep everyone honest), the file matrix now displays the folks responsible for the latest review marks:
Between these two new features, it's now easier to reach the “review complete” state even if a reviewer forgot to mark a file as reviewed or acknowledge a comment. And that lets you take advantage of the recently added Merge Branch feature, so you don't need to go back to GitHub (or your shell) to get your code merged in. Winning!