)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000010,"name":"Mateusz Zalega","display_name":"msgctl","email":"mateusz@monogon.tech","username":"mateusz","avatars":[{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"ced4c7ae58dde8207f9523c3121e67e8ea360ead","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"202b3cba_c0ad7d4a","updated":"2022-07-14 16:31:48.000000000","message":"Shall it go in, or is it an unnecessary CPU burden?","commit_id":"0acd6acf4e1a1a10ba3bfe89d9879e5e5d6110b1"}],"metropolis/cli/metroctl/test/test.go":[{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"acc835d3e2cd0bd4f1d5090c4987b1961c98be59","unresolved":true,"context_lines":[{"line_number":26,"context_line":"\t\treturn fmt.Errorf(\"Couldn\u0027t resolve metroctl binary: %v\", err)"},{"line_number":27,"context_line":"\t}"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"\tlog.Printf(\"metroctl %s\", strings.Join(args, \" \"))"},{"line_number":30,"context_line":"\tfound, err :\u003d cmd.RunCommand(ctx, path, args, expect)"},{"line_number":31,"context_line":"\tif err !\u003d nil {"},{"line_number":32,"context_line":"\t\treturn fmt.Errorf(\"While running metroctl: %v\", err)"}],"source_content_type":"text/x-go","patch_set":4,"id":"41410033_2b55aa9f","line":29,"updated":"2022-08-04 13:47:04.000000000","message":"Maybe prefix the \u0027commands we\u0027re executing\u0027 logs with a `$`? That seems like an okay convention to distinguish commands from output/expected output/etc.","commit_id":"0acd6acf4e1a1a10ba3bfe89d9879e5e5d6110b1"},{"author":{"_account_id":1000010,"name":"Mateusz Zalega","display_name":"msgctl","email":"mateusz@monogon.tech","username":"mateusz","avatars":[{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"4268b4144c7fc11e230e7df79ac5d98a641bf62b","unresolved":false,"context_lines":[{"line_number":26,"context_line":"\t\treturn fmt.Errorf(\"Couldn\u0027t resolve metroctl binary: %v\", err)"},{"line_number":27,"context_line":"\t}"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"\tlog.Printf(\"metroctl %s\", strings.Join(args, \" \"))"},{"line_number":30,"context_line":"\tfound, err :\u003d cmd.RunCommand(ctx, path, args, expect)"},{"line_number":31,"context_line":"\tif err !\u003d nil {"},{"line_number":32,"context_line":"\t\treturn fmt.Errorf(\"While running metroctl: %v\", err)"}],"source_content_type":"text/x-go","patch_set":4,"id":"d1b5605a_2930a023","line":29,"in_reply_to":"41410033_2b55aa9f","updated":"2022-08-08 15:06:03.000000000","message":"Done","commit_id":"0acd6acf4e1a1a10ba3bfe89d9879e5e5d6110b1"},{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"acc835d3e2cd0bd4f1d5090c4987b1961c98be59","unresolved":true,"context_lines":[{"line_number":29,"context_line":"\tlog.Printf(\"metroctl %s\", strings.Join(args, \" \"))"},{"line_number":30,"context_line":"\tfound, err :\u003d cmd.RunCommand(ctx, path, args, expect)"},{"line_number":31,"context_line":"\tif err !\u003d nil {"},{"line_number":32,"context_line":"\t\treturn fmt.Errorf(\"While running metroctl: %v\", err)"},{"line_number":33,"context_line":"\t}"},{"line_number":34,"context_line":"\tif !found {"},{"line_number":35,"context_line":"\t\treturn fmt.Errorf(\"Expected string wasn\u0027t found while running metroctl.\")"}],"source_content_type":"text/x-go","patch_set":4,"id":"e618ac55_c5e99a16","line":32,"range":{"start_line":32,"start_character":21,"end_line":32,"end_character":22},"updated":"2022-08-04 13:47:04.000000000","message":"Errors start with a lowercase letter.","commit_id":"0acd6acf4e1a1a10ba3bfe89d9879e5e5d6110b1"},{"author":{"_account_id":1000010,"name":"Mateusz Zalega","display_name":"msgctl","email":"mateusz@monogon.tech","username":"mateusz","avatars":[{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/30cae8ca0782f23ce0a60ac80fda3dd9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"4268b4144c7fc11e230e7df79ac5d98a641bf62b","unresolved":false,"context_lines":[{"line_number":29,"context_line":"\tlog.Printf(\"metroctl %s\", strings.Join(args, \" \"))"},{"line_number":30,"context_line":"\tfound, err :\u003d cmd.RunCommand(ctx, path, args, expect)"},{"line_number":31,"context_line":"\tif err !\u003d nil {"},{"line_number":32,"context_line":"\t\treturn fmt.Errorf(\"While running metroctl: %v\", err)"},{"line_number":33,"context_line":"\t}"},{"line_number":34,"context_line":"\tif !found {"},{"line_number":35,"context_line":"\t\treturn fmt.Errorf(\"Expected string wasn\u0027t found while running metroctl.\")"}],"source_content_type":"text/x-go","patch_set":4,"id":"e1551aa4_21ec4e18","line":32,"range":{"start_line":32,"start_character":21,"end_line":32,"end_character":22},"in_reply_to":"e618ac55_c5e99a16","updated":"2022-08-08 15:06:03.000000000","message":"Done","commit_id":"0acd6acf4e1a1a10ba3bfe89d9879e5e5d6110b1"}]}