)]}'
{"cmd/cluster/initinfra.go":[{"author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"change_message_id":"62b9d74299a280d7b32d7ef385f9e098dc5ed37f","unresolved":false,"context_lines":[{"line_number":70,"context_line":"\t\t\u0026i.ClusterType,"},{"line_number":71,"context_line":"\t\t\"cluster-type\","},{"line_number":72,"context_line":"\t\t\"ephemeral\","},{"line_number":73,"context_line":"\t\t`select cluster type to deploy initial infastructure to;`+"},{"line_number":74,"context_line":"\t\t\t` currently only ephemeral is supported`)"},{"line_number":75,"context_line":"}"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_596a3480","line":73,"range":{"start_line":73,"start_character":41,"end_line":73,"end_character":54},"updated":"2020-04-21 17:09:54.000000000","message":"good opportunity to address spelling here","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"66369d0c93d16523db1355c169d88f58dceb2b0f","unresolved":false,"context_lines":[{"line_number":70,"context_line":"\t\t\u0026i.ClusterType,"},{"line_number":71,"context_line":"\t\t\"cluster-type\","},{"line_number":72,"context_line":"\t\t\"ephemeral\","},{"line_number":73,"context_line":"\t\t`select cluster type to deploy initial infastructure to;`+"},{"line_number":74,"context_line":"\t\t\t` currently only ephemeral is supported`)"},{"line_number":75,"context_line":"}"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_0a384cd1","line":73,"range":{"start_line":73,"start_character":41,"end_line":73,"end_character":54},"in_reply_to":"1f493fa4_596a3480","updated":"2020-04-21 18:39:42.000000000","message":"Done","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"}],"cmd/config/init.go":[{"author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"change_message_id":"62b9d74299a280d7b32d7ef385f9e098dc5ed37f","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"// NewInitCommand creates a command for generating default airshipctl config files."},{"line_number":36,"context_line":"func NewInitCommand(rootSettings *environment.AirshipCTLSettings) *cobra.Command {"},{"line_number":37,"context_line":"\t// TODO(howell): It\u0027d be nice to have an --output flag to tell"},{"line_number":38,"context_line":"\t// airshipctl where to store the new files."},{"line_number":39,"context_line":"\t// TODO(howell): Currently, this command overwrites whatever the user"},{"line_number":40,"context_line":"\t// has in their airship directory. We should remove that functionality"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_99183cd4","line":37,"range":{"start_line":37,"start_character":42,"end_line":37,"end_character":50},"updated":"2020-04-21 17:09:54.000000000","message":"observation: from Pegleg experience, we\u0027ll want to standardize a \"save location\" type variable across all commands.  mixing --output and --save-location down the road will be a headache.","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"66369d0c93d16523db1355c169d88f58dceb2b0f","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"// NewInitCommand creates a command for generating default airshipctl config files."},{"line_number":36,"context_line":"func NewInitCommand(rootSettings *environment.AirshipCTLSettings) *cobra.Command {"},{"line_number":37,"context_line":"\t// TODO(howell): It\u0027d be nice to have an --output flag to tell"},{"line_number":38,"context_line":"\t// airshipctl where to store the new files."},{"line_number":39,"context_line":"\t// TODO(howell): Currently, this command overwrites whatever the user"},{"line_number":40,"context_line":"\t// has in their airship directory. We should remove that functionality"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_aa75d89a","line":37,"range":{"start_line":37,"start_character":42,"end_line":37,"end_character":50},"in_reply_to":"1f493fa4_99183cd4","updated":"2020-04-21 18:39:42.000000000","message":"good call - I\u0027ll remove the name suggestion to prevent future confusion","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"change_message_id":"62b9d74299a280d7b32d7ef385f9e098dc5ed37f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"\t// airshipctl where to store the new files."},{"line_number":39,"context_line":"\t// TODO(howell): Currently, this command overwrites whatever the user"},{"line_number":40,"context_line":"\t// has in their airship directory. We should remove that functionality"},{"line_number":41,"context_line":"\t// as default and provide and optional --overwrite flag."},{"line_number":42,"context_line":"\tcmd :\u003d \u0026cobra.Command{"},{"line_number":43,"context_line":"\t\tUse:   \"init\","},{"line_number":44,"context_line":"\t\tShort: \"Generate initial configuration files for airshipctl\","}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_997ddce9","line":41,"range":{"start_line":41,"start_character":7,"end_line":41,"end_character":14},"updated":"2020-04-21 17:09:54.000000000","message":"what would default behavior be then?  require a specified save location?","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"66369d0c93d16523db1355c169d88f58dceb2b0f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"\t// airshipctl where to store the new files."},{"line_number":39,"context_line":"\t// TODO(howell): Currently, this command overwrites whatever the user"},{"line_number":40,"context_line":"\t// has in their airship directory. We should remove that functionality"},{"line_number":41,"context_line":"\t// as default and provide and optional --overwrite flag."},{"line_number":42,"context_line":"\tcmd :\u003d \u0026cobra.Command{"},{"line_number":43,"context_line":"\t\tUse:   \"init\","},{"line_number":44,"context_line":"\t\tShort: \"Generate initial configuration files for airshipctl\","}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_0a668cee","line":41,"range":{"start_line":41,"start_character":7,"end_line":41,"end_character":14},"in_reply_to":"1f493fa4_997ddce9","updated":"2020-04-21 18:39:42.000000000","message":"I would think an error - \"Could not write $HOME/.airship/config - file exists\" or something similar.\n\nEither way, this needs more design and is pretty low priority right now","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"}],"cmd/config/set_authinfo.go":[{"author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"change_message_id":"62b9d74299a280d7b32d7ef385f9e098dc5ed37f","unresolved":false,"context_lines":[{"line_number":29,"context_line":"\tsetAuthInfoLong \u003d `"},{"line_number":30,"context_line":"Create or modify a user credential in the airshipctl config file."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Note that a user may not have more than one authentication method at the same time."},{"line_number":33,"context_line":"`"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"\tsetAuthInfoExample \u003d `"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_794f30bd","line":32,"range":{"start_line":32,"start_character":0,"end_line":32,"end_character":83},"updated":"2020-04-21 17:09:54.000000000","message":"is an error thrown to prevent multiple auth methods, or is the previous auth method replaced if a conflicting one is added?","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"66369d0c93d16523db1355c169d88f58dceb2b0f","unresolved":false,"context_lines":[{"line_number":29,"context_line":"\tsetAuthInfoLong \u003d `"},{"line_number":30,"context_line":"Create or modify a user credential in the airshipctl config file."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Note that a user may not have more than one authentication method at the same time."},{"line_number":33,"context_line":"`"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"\tsetAuthInfoExample \u003d `"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_6ab8f029","line":32,"range":{"start_line":32,"start_character":0,"end_line":32,"end_character":83},"in_reply_to":"1f493fa4_794f30bd","updated":"2020-04-21 18:39:42.000000000","message":"Updated to match the code","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"}],"cmd/config/set_cluster.go":[{"author":{"_account_id":29624,"name":"Alexander Hughes","email":"Alexander.Hughes@pm.me","username":"alexanderhughes"},"change_message_id":"62b9d74299a280d7b32d7ef385f9e098dc5ed37f","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  --cluster-type\u003dephemeral \\"},{"line_number":41,"context_line":"  --server\u003dhttps://1.2.3.4"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"# Embed certificate authority data for the target exampleCluser"},{"line_number":44,"context_line":"airshipctl config set-cluster exampleCluster \\"},{"line_number":45,"context_line":"  --cluster-type\u003dtarget \\"},{"line_number":46,"context_line":"  --client-certificate-authority\u003d$HOME/.airship/ca/kubernetes.ca.crt \\"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_99cb1c1d","line":43,"range":{"start_line":43,"start_character":57,"end_line":43,"end_character":63},"updated":"2020-04-21 17:09:54.000000000","message":"Cluster","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"66369d0c93d16523db1355c169d88f58dceb2b0f","unresolved":false,"context_lines":[{"line_number":40,"context_line":"  --cluster-type\u003dephemeral \\"},{"line_number":41,"context_line":"  --server\u003dhttps://1.2.3.4"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"# Embed certificate authority data for the target exampleCluser"},{"line_number":44,"context_line":"airshipctl config set-cluster exampleCluster \\"},{"line_number":45,"context_line":"  --cluster-type\u003dtarget \\"},{"line_number":46,"context_line":"  --client-certificate-authority\u003d$HOME/.airship/ca/kubernetes.ca.crt \\"}],"source_content_type":"text/x-go","patch_set":5,"id":"1f493fa4_caf784f5","line":43,"range":{"start_line":43,"start_character":57,"end_line":43,"end_character":63},"in_reply_to":"1f493fa4_99cb1c1d","updated":"2020-04-21 18:39:42.000000000","message":"Done","commit_id":"d15689c1b2fc794484f1bdaa62ec9ffd1cc21a86"}]}