)]}'
{"cbmgr.py":[{"author":{"_account_id":1004460,"name":"James Lee","email":"james.lee@couchbase.com","username":"jamesl33","avatars":[{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"5cb168bcb8df76c4028e72aabbff1c2ecde657e5","unresolved":true,"context_lines":[{"line_number":1637,"context_line":"            [\u0027dist_config_file\u0027, f\u0027\"{dist_cfg_file}\"\u0027, \u0027-setcookie\u0027, cookie, \u0027-run\u0027, \u0027encryption_service\u0027,"},{"line_number":1638,"context_line":"             \u0027remote_set_password\u0027, node]"},{"line_number":1639,"context_line":""},{"line_number":1640,"context_line":"        rc, out, err \u003d self.run_process(\"erl\", args, {\u0027setpassword\u0027: password})"},{"line_number":1641,"context_line":""},{"line_number":1642,"context_line":"        if rc \u003d\u003d 0:"},{"line_number":1643,"context_line":"            print(\"SUCCESS: Password accepted. Node started booting.\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"fb3288c7_e6722ee8","line":1640,"range":{"start_line":1640,"start_character":55,"end_line":1640,"end_character":66},"updated":"2021-10-12 13:07:59.000000000","message":"NITPICKING: Isn\u0027t the convention to use all uppercase for environment variables?","commit_id":"4f17fd5445a857fe8306964281c03f8f890dbd22"},{"author":{"_account_id":1004257,"name":"Timofey Barmin","email":"timofey.barmin@couchbase.com","username":"timofey-barmin","avatars":[{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"ca6e405ff9a01474fbb2ac1356f1661213905708","unresolved":true,"context_lines":[{"line_number":1637,"context_line":"            [\u0027dist_config_file\u0027, f\u0027\"{dist_cfg_file}\"\u0027, \u0027-setcookie\u0027, cookie, \u0027-run\u0027, \u0027encryption_service\u0027,"},{"line_number":1638,"context_line":"             \u0027remote_set_password\u0027, node]"},{"line_number":1639,"context_line":""},{"line_number":1640,"context_line":"        rc, out, err \u003d self.run_process(\"erl\", args, {\u0027setpassword\u0027: password})"},{"line_number":1641,"context_line":""},{"line_number":1642,"context_line":"        if rc \u003d\u003d 0:"},{"line_number":1643,"context_line":"            print(\"SUCCESS: Password accepted. Node started booting.\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"b0624960_e0806e7f","line":1640,"range":{"start_line":1640,"start_character":55,"end_line":1640,"end_character":66},"in_reply_to":"fb3288c7_e6722ee8","updated":"2021-10-12 17:06:41.000000000","message":"done","commit_id":"4f17fd5445a857fe8306964281c03f8f890dbd22"},{"author":{"_account_id":1004460,"name":"James Lee","email":"james.lee@couchbase.com","username":"jamesl33","avatars":[{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52d2998b3f81f6ca2acd4a81886a9a35.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"5cb168bcb8df76c4028e72aabbff1c2ecde657e5","unresolved":true,"context_lines":[{"line_number":1653,"context_line":"        else:"},{"line_number":1654,"context_line":"            _exit_if_errors([f\u0027Unknown error: {rc} {out}, {err}\u0027])"},{"line_number":1655,"context_line":""},{"line_number":1656,"context_line":"    def run_process(self, name, args, extra_env):"},{"line_number":1657,"context_line":"        try:"},{"line_number":1658,"context_line":"            if os.name \u003d\u003d \"nt\":"},{"line_number":1659,"context_line":"                name \u003d name + \".exe\""}],"source_content_type":"text/x-python","patch_set":2,"id":"2b0d0430_97bb89ad","line":1656,"range":{"start_line":1656,"start_character":38,"end_line":1656,"end_character":47},"updated":"2021-10-12 13:07:59.000000000","message":"NITPICKING: Perhaps it would be safer to give this a default value, this would avoid calls elsewhere failing (there may not be any, but might be worth thinking about)","commit_id":"4f17fd5445a857fe8306964281c03f8f890dbd22"},{"author":{"_account_id":1004257,"name":"Timofey Barmin","email":"timofey.barmin@couchbase.com","username":"timofey-barmin","avatars":[{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/f3d67f253451b9d3d488bd28a7020b53.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"ca6e405ff9a01474fbb2ac1356f1661213905708","unresolved":true,"context_lines":[{"line_number":1653,"context_line":"        else:"},{"line_number":1654,"context_line":"            _exit_if_errors([f\u0027Unknown error: {rc} {out}, {err}\u0027])"},{"line_number":1655,"context_line":""},{"line_number":1656,"context_line":"    def run_process(self, name, args, extra_env):"},{"line_number":1657,"context_line":"        try:"},{"line_number":1658,"context_line":"            if os.name \u003d\u003d \"nt\":"},{"line_number":1659,"context_line":"                name \u003d name + \".exe\""}],"source_content_type":"text/x-python","patch_set":2,"id":"78af5564_ea4422ca","line":1656,"range":{"start_line":1656,"start_character":38,"end_line":1656,"end_character":47},"in_reply_to":"2b0d0430_97bb89ad","updated":"2021-10-12 17:06:41.000000000","message":"makes sense, done","commit_id":"4f17fd5445a857fe8306964281c03f8f890dbd22"}]}