)]}'
{"tempest/cmd/run.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b2daa8cc30fa7fac8d5c7f28175faf50369b7bfe","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            if not os.path.isdir(\u0027.testrepository\u0027):"},{"line_number":79,"context_line":"                returncode \u003d run_argv([\u0027testr\u0027, \u0027init\u0027], sys.stdin, sys.stdout,"},{"line_number":80,"context_line":"                                      sys.stderr)"},{"line_number":81,"context_line":"            if returncode:"},{"line_number":82,"context_line":"                sys.exit(returncode)"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            print(\"No .testr.conf file was found for local execution\")"},{"line_number":85,"context_line":"            sys.exit(2)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_da9cd66b","line":82,"range":{"start_line":81,"start_character":0,"end_line":82,"end_character":36},"updated":"2016-06-22 08:34:04.000000000","message":"Also this condition should be inside \"if not os.path.isdir(\u0027.testrepository\u0027):\"\n\nto avoid unnecessary checking of returncode when testrepository exist.","commit_id":"550093904aa2d81e06643b374bfaabb32ae15d05"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93e1f9424f910d0291c3769f95d4fa8a17e03f86","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            if not os.path.isdir(\u0027.testrepository\u0027):"},{"line_number":79,"context_line":"                returncode \u003d run_argv([\u0027testr\u0027, \u0027init\u0027], sys.stdin, sys.stdout,"},{"line_number":80,"context_line":"                                      sys.stderr)"},{"line_number":81,"context_line":"            if returncode:"},{"line_number":82,"context_line":"                sys.exit(returncode)"},{"line_number":83,"context_line":"        else:"},{"line_number":84,"context_line":"            print(\"No .testr.conf file was found for local execution\")"},{"line_number":85,"context_line":"            sys.exit(2)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_9a236e05","line":82,"range":{"start_line":81,"start_character":0,"end_line":82,"end_character":36},"in_reply_to":"3aaa91ec_da9cd66b","updated":"2016-06-22 08:34:53.000000000","message":"this was causing the error.","commit_id":"550093904aa2d81e06643b374bfaabb32ae15d05"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"ba361815a15f43e1b91becc4b23becfe82c6cd14","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def take_action(self, parsed_args):"},{"line_number":72,"context_line":"        self._set_env()"},{"line_number":73,"context_line":"        returncode \u003d 0"},{"line_number":74,"context_line":"        # Local execution mode"},{"line_number":75,"context_line":"        if os.path.isfile(\u0027.testr.conf\u0027):"},{"line_number":76,"context_line":"            # If you\u0027re running in local execution mode and there is not a"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_9dfdfa3c","line":73,"range":{"start_line":73,"start_character":8,"end_line":73,"end_character":22},"updated":"2016-06-22 23:01:29.000000000","message":"This is not strictly necessary since the change in L81/82, but it doesn\u0027t harm either.","commit_id":"fe2fa00e13c23d5529cb16bc08647d0599709297"}]}