Fix up the Rook valid_moves() and go() routines
Convert Rook to new generator-based valid_moves()
Remove old Rook.is_valid_move(); much celebration!
Move valid_moves() method to ChessPiece
Refactor Bishop to use new valid_moves()
Add TODO list
Tell git to ignore "coverage" tool output