Entertainer Media Center

Code review comment for lp:~rockstar/entertainer/kill-fixmes

  1. kill-fixmes
  2. Merge into trunk
Revision history for this message
Matt Layman (mblayman) wrote :

 approve merge_conditional

This branch looks fine, but I have a couple of comments about the pylintrc. You've removed W0511 but still have the XXX comment.

I see that you removed TODO and XXX from the things to check. Do you think you could add an XXX comment indicating that we plan on adding TODO back to the list of things to check.

review: Approve

« Back to merge proposal