)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"7aa322ce2564c53f7585e13fae11d568683fffe4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove NotImplementedError exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch aim to change \u0027NotImplementedError\u0027 to \u0027pass\u0027 statement into"},{"line_number":10,"context_line":"methods that already implemented"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bf689b6b31036ff8dea522fc90f8e086b726cdf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_dd9944d0","line":10,"updated":"2018-06-08 12:34:43.000000000","message":"Why?","commit_id":"eceeae9a54f13a6aecdf88107f2e4c4d3febe798"},{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"b2b774b8ec729ad1f3ab0e07b23bcb7c33950e4e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove NotImplementedError exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch aim to change \u0027NotImplementedError\u0027 to \u0027pass\u0027 statement into"},{"line_number":10,"context_line":"methods that already implemented"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bf689b6b31036ff8dea522fc90f8e086b726cdf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_7d7d821a","line":10,"in_reply_to":"5f7c97a3_54e66a18","updated":"2018-06-09 06:48:21.000000000","message":"Sure, but why even add “pass”. If there is a docstring it isn’t needed and any method like this should have a docstring.","commit_id":"eceeae9a54f13a6aecdf88107f2e4c4d3febe798"},{"author":{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},"change_message_id":"bf3e61dfa8932f62f061402f6273fcbad7781263","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove NotImplementedError exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch aim to change \u0027NotImplementedError\u0027 to \u0027pass\u0027 statement into"},{"line_number":10,"context_line":"methods that already implemented"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bf689b6b31036ff8dea522fc90f8e086b726cdf"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_54e66a18","line":10,"in_reply_to":"5f7c97a3_dd9944d0","updated":"2018-06-08 18:32:15.000000000","message":"Because it doesn\u0027t make sense. These methods will throw \u0027Can\u0027t instantiate abstract class\u0027 error anyway","commit_id":"eceeae9a54f13a6aecdf88107f2e4c4d3febe798"}],"mistral/engine/actions.py":[{"author":{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},"change_message_id":"bf3e61dfa8932f62f061402f6273fcbad7781263","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        \"\"\""},{"line_number":137,"context_line":"        pass"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def validate_input(self, input_dict):"},{"line_number":140,"context_line":"        \"\"\"Validates action input parameters."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        :param input_dict: Dictionary with input parameters."}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_34c2ce54","line":139,"updated":"2018-06-08 18:32:15.000000000","message":"Func must has the @abc.abstractmethod decorator","commit_id":"eceeae9a54f13a6aecdf88107f2e4c4d3febe798"}]}