)]}'
{"mistral/tests/unit/engine/test_adhoc_actions.py":[{"author":{"_account_id":23630,"name":"Tovin Seven","email":"tovin07@gmail.com","username":"tovin07"},"change_message_id":"de39e8adbfd1b926e7b12fcffc4d97d0faa38c6e","unresolved":false,"context_lines":[{"line_number":214,"context_line":""},{"line_number":215,"context_line":"    def test_run_adhoc_action_with_env(self):"},{"line_number":216,"context_line":"        wf_ex \u003d self.engine.start_workflow("},{"line_number":217,"context_line":"            \u0027my_wb.wf4\u0027,"},{"line_number":218,"context_line":"            \u0027\u0027,"},{"line_number":219,"context_line":"            {\u0027str1\u0027: \u0027a\u0027},"},{"line_number":220,"context_line":"            env\u003d{\u0027foo\u0027: \u0027bar\u0027}"},{"line_number":221,"context_line":"        )"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        self.await_workflow_success(wf_ex.id)"},{"line_number":224,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ff82abbf_12bcd8da","line":221,"range":{"start_line":217,"start_character":0,"end_line":221,"end_character":9},"updated":"2017-11-21 08:13:07.000000000","message":"why?","commit_id":"fe3c7bd1977da41a0eb240c3131e2e5cf79acd9a"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"dfe64ff703fabfb408fc00326d8acb5100387d5e","unresolved":false,"context_lines":[{"line_number":214,"context_line":""},{"line_number":215,"context_line":"    def test_run_adhoc_action_with_env(self):"},{"line_number":216,"context_line":"        wf_ex \u003d self.engine.start_workflow("},{"line_number":217,"context_line":"            \u0027my_wb.wf4\u0027,"},{"line_number":218,"context_line":"            \u0027\u0027,"},{"line_number":219,"context_line":"            {\u0027str1\u0027: \u0027a\u0027},"},{"line_number":220,"context_line":"            env\u003d{\u0027foo\u0027: \u0027bar\u0027}"},{"line_number":221,"context_line":"        )"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        self.await_workflow_success(wf_ex.id)"},{"line_number":224,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ff82abbf_18287984","line":221,"range":{"start_line":217,"start_character":0,"end_line":221,"end_character":9},"in_reply_to":"ff82abbf_12bcd8da","updated":"2017-11-21 09:11:28.000000000","message":"Tovin, not sure what you\u0027re asking here.. But I have to admit that people on the project do that mostly because of my whim. I encourage them to use blank lines as separators of different logical units in the code. For example, line 223 here is awaiting for workflow success. It has nothing to do with the block 225-233 and should be separated somehow so that we see that we started something else.\n\nHowever, if you meant differently formatted method call then I believe that this format reads well if we have a long list of arguments that doesn\u0027t fit to 80 chars. Because we clearly see each method argument on a separate line. It\u0027s not a strict requirement though to do so, we do both ways across the code.","commit_id":"fe3c7bd1977da41a0eb240c3131e2e5cf79acd9a"},{"author":{"_account_id":21970,"name":"Andras Kovi","email":"akovi@nokia.com","username":"andras.kovi"},"change_message_id":"5f43ab056611115a9519c8b559a1e5b404033dcb","unresolved":false,"context_lines":[{"line_number":307,"context_line":""},{"line_number":308,"context_line":"        wf_ex \u003d self.engine.start_workflow(\u0027my_wb1.my_wf\u0027)"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"        self.await_workflow_running(wf_ex.id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"df87a7cf_8fc27833","line":310,"range":{"start_line":310,"start_character":8,"end_line":310,"end_character":45},"updated":"2017-12-08 12:20:57.000000000","message":"A note about why RUNNING is adequate instead of a terminal state would be useful.","commit_id":"60a8847196f0552c7561b0b0942b4f8c3ac7836b"}]}