)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"1e5e0cc961af89a536a82b21d0c7abc9f76d0065","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Missing space for pragma cover"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patchset add space to ignore abstract in test"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I7863a8ba7e20358fb7cdf3cc7e4d83871a5104ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a740942_542ea0fb","line":9,"updated":"2015-12-08 23:15:12.000000000","message":"What test?","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"cac704b984c16121db7c8628da048c9e13bc2f09","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Missing space for pragma cover"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patchset add space to ignore abstract in test"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I7863a8ba7e20358fb7cdf3cc7e4d83871a5104ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a740942_9d0add24","line":9,"in_reply_to":"7a740942_542ea0fb","updated":"2015-12-09 08:25:29.000000000","message":"Did you mean \"coverage\" instead of test?","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"fd13a3c75d52ab36e12d3b8d1c812a0b612bf9fd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Missing space for pragma cover"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patchset add space to ignore abstract in test"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I7863a8ba7e20358fb7cdf3cc7e4d83871a5104ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a740942_cf42e80e","line":9,"in_reply_to":"7a740942_9d0add24","updated":"2015-12-09 08:30:51.000000000","message":"yes :)","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"}],"watcher/decision_engine/solution/solution_comparator.py":[{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"1e5e0cc961af89a536a82b21d0c7abc9f76d0065","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class Solution(object):"},{"line_number":25,"context_line":"    @abc.abstractmethod"},{"line_number":26,"context_line":"    def compare(self, sol1, sol2):"},{"line_number":27,"context_line":"        raise NotImplementedError("},{"line_number":28,"context_line":"            \"Should have implemented this\")  # pragma: no cover"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_14981869","line":27,"updated":"2015-12-08 23:15:12.000000000","message":"Why not just use `pass` instead?  An exception will already be raised if someone tries to instantiate a subclass of Solution without implementing any of the methods annotated with @abstractmethod.","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"cac704b984c16121db7c8628da048c9e13bc2f09","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class Solution(object):"},{"line_number":25,"context_line":"    @abc.abstractmethod"},{"line_number":26,"context_line":"    def compare(self, sol1, sol2):"},{"line_number":27,"context_line":"        raise NotImplementedError("},{"line_number":28,"context_line":"            \"Should have implemented this\")  # pragma: no cover"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_8f3860d4","line":27,"in_reply_to":"7a740942_14981869","updated":"2015-12-09 08:25:29.000000000","message":"True, but then you would still need to put the \"pragma: no cover\" thing to exclude this line from the coverage score.","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"}],"watcher/decision_engine/solution/solution_evaluator.py":[{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"1e5e0cc961af89a536a82b21d0c7abc9f76d0065","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class SolutionEvaluator(object):"},{"line_number":25,"context_line":"    @abc.abstractmethod"},{"line_number":26,"context_line":"    def evaluate(self, solution):"},{"line_number":27,"context_line":"        raise NotImplementedError("},{"line_number":28,"context_line":"            \"Should have implemented this\")  # pragma: no cover"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_94ac088d","line":27,"updated":"2015-12-08 23:15:12.000000000","message":"Ditto","commit_id":"fd399a09d3fd1ba61738800aa66c3e8b85e33b0e"}]}