)]}'
{"cliff/command.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f2a9b7806dae67868c7e611957d10b51c7af21f2","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    def get_epilog(self):"},{"line_number":91,"context_line":"        \"\"\"Return the command epilog.\"\"\""},{"line_number":92,"context_line":"        hook_epilogs \u003d filter("},{"line_number":93,"context_line":"            None,"},{"line_number":94,"context_line":"            (h.obj.get_epilog() for h in self._hooks),"},{"line_number":95,"context_line":"        )"},{"line_number":96,"context_line":"        if hook_epilogs:"},{"line_number":97,"context_line":"            # combine them, replacing a None in self._epilog with an"},{"line_number":98,"context_line":"            # empty string"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf091321_bf13a9b8","line":95,"range":{"start_line":92,"start_character":23,"end_line":95,"end_character":9},"updated":"2017-06-08 09:53:01.000000000","message":"nit: I\u0027d personally find a list comprehension a little easier to read here","commit_id":"aa65cc1bba7390774b50c1641001a67eb7cb7a2c"}],"cliff/hooks.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"624a316478e59b3596d31eff71a1b906dead2d8a","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        \"\"\""},{"line_number":38,"context_line":"        return parser"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @abc.abstractmethod"},{"line_number":41,"context_line":"    def get_epilog(self):"},{"line_number":42,"context_line":"        \"Return text to add to the command help epilog.\""},{"line_number":43,"context_line":"        return \u0027\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f231b9d_36741f68","line":40,"range":{"start_line":40,"start_character":4,"end_line":40,"end_character":23},"updated":"2017-06-13 13:37:49.000000000","message":"This requires all hooks to define get_epilog.\nAccording to cliff.command implementation, None is ignored.\nHow about returning None by default?","commit_id":"0933c49a7c754dd90d66bc22f281d7e54f76581f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"017f96206c90cdbb0faf8fbad932d947d2e66879","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        \"\"\""},{"line_number":38,"context_line":"        return parser"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @abc.abstractmethod"},{"line_number":41,"context_line":"    def get_epilog(self):"},{"line_number":42,"context_line":"        \"Return text to add to the command help epilog.\""},{"line_number":43,"context_line":"        return \u0027\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f231b9d_c51d2810","line":40,"range":{"start_line":40,"start_character":4,"end_line":40,"end_character":23},"in_reply_to":"7f231b9d_36741f68","updated":"2017-06-13 17:30:11.000000000","message":"I would prefer to have hook authors explicitly implement each hook rather than relying on base implementations.","commit_id":"0933c49a7c754dd90d66bc22f281d7e54f76581f"}],"cliff/tests/test_command_hooks.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f2a9b7806dae67868c7e611957d10b51c7af21f2","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        self.assertEqual(\u0027cliff.tests.test\u0027, kwargs[\u0027namespace\u0027])"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"class TestHooks(base.TestBase):"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def setUp(self):"},{"line_number":92,"context_line":"        super(TestHooks, self).setUp()"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf091321_1f24d50c","line":89,"range":{"start_line":89,"start_character":6,"end_line":89,"end_character":15},"updated":"2017-06-08 09:53:01.000000000","message":"This could probably be rolled back into the previous patch","commit_id":"aa65cc1bba7390774b50c1641001a67eb7cb7a2c"}]}