)]}'
{"mistral/workbook/v2/workflows.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"7691089ae2201ae4fd6732603b25e30c5779bd16","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        # from mistral.workbook.parser."},{"line_number":84,"context_line":"        from mistral.workflow import commands"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        if (task_name not in commands.RESERVED_CMDS"},{"line_number":87,"context_line":"                and not self._task_exists(task_name)):"},{"line_number":88,"context_line":"            raise exc.InvalidModelException("},{"line_number":89,"context_line":"                \"Task \u0027%s\u0027 not found.\" % task_name"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba15a1d1_4f40ea56","line":86,"updated":"2015-09-15 08:28:57.000000000","message":"What if we make this list of special commands a part of workbook module somewhere? Logically, it does look right to me because it\u0027s also a part of the schema. In this case we wouldn\u0027t have this circular dependency.","commit_id":"df3a64c51f85a0df49459d1f6b990c66b0f0ff60"}],"mistral/workflow/commands.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"5c99c9deea630f6ef43d68d1c15f89345582ec3e","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return \"Pause [workflow\u003d%s]\" % self.wf_ex.name"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"RESERVED_CMDS \u003d dict(zip("},{"line_number":135,"context_line":"    tasks.RESERVED_TASK_NAMES, ["},{"line_number":136,"context_line":"        Noop,"},{"line_number":137,"context_line":"        FailWorkflow,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba15a1d1_924037a7","line":134,"updated":"2015-09-15 15:21:27.000000000","message":"Soft: looks a little bit fragile. If we change an order in one of these structures it will break the system. On the other hand, if we just use string literals here it won\u0027t be any better because we may change their values themselves at some point (not so likely). So I think it\u0027s ok.","commit_id":"705bbba2760d670b196a18df5fb6e762d9647ad4"},{"author":{"_account_id":7700,"name":"Nikolay Mahotkin","email":"nikolay.makhotkin@gmail.com","username":"nmakhotkin"},"change_message_id":"298e1047ed478bc096d27d73b393255492e4ea79","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return \"Pause [workflow\u003d%s]\" % self.wf_ex.name"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"RESERVED_CMDS \u003d dict(zip("},{"line_number":135,"context_line":"    tasks.RESERVED_TASK_NAMES, ["},{"line_number":136,"context_line":"        Noop,"},{"line_number":137,"context_line":"        FailWorkflow,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba15a1d1_dc780192","line":134,"in_reply_to":"ba15a1d1_924037a7","updated":"2015-09-15 16:35:21.000000000","message":"we can sort the list alphabetically before we create this dict.","commit_id":"705bbba2760d670b196a18df5fb6e762d9647ad4"}]}