)]}'
{"cmd/baremetal/baremetal.go":[{"author":{"_account_id":24580,"name":"James Gu","email":"james.gu@microsoft.com","username":"jamesgu"},"change_message_id":"0c0babf6394a14cad54f65dcce0cd207b658b5c6","unresolved":false,"context_lines":[{"line_number":72,"context_line":"\treturn baremetalRootCmd"},{"line_number":73,"context_line":"}"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"// GetHostSelections builds a list of selectors that can be passed to a manager using the name and label flags passed to"},{"line_number":76,"context_line":"// airshipctl."},{"line_number":77,"context_line":"func GetHostSelections(name string, labels string) []remote.HostSelector {"},{"line_number":78,"context_line":"\tvar selectors []remote.HostSelector"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_142ca500","line":75,"updated":"2020-05-14 15:21:33.000000000","message":"is the line too long?","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"},{"author":{"_account_id":28690,"name":"Sirajudeen","email":"sirajudeen.yasin@gmail.com","username":"siraj.yasin"},"change_message_id":"9a2c6477f6d976dd0601c11539c8cbddd1997338","unresolved":false,"context_lines":[{"line_number":72,"context_line":"\treturn baremetalRootCmd"},{"line_number":73,"context_line":"}"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"// GetHostSelections builds a list of selectors that can be passed to a manager using the name and label flags passed to"},{"line_number":76,"context_line":"// airshipctl."},{"line_number":77,"context_line":"func GetHostSelections(name string, labels string) []remote.HostSelector {"},{"line_number":78,"context_line":"\tvar selectors []remote.HostSelector"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_6c227554","line":75,"in_reply_to":"ff570b3c_142ca500","updated":"2020-05-14 17:11:08.000000000","message":"\u003e is the line too long?\n\nI think 120 is allowed for comment. But then i broke some part of it to next line as we had space to accommodate in next line.","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"}],"pkg/k8s/kubectl/interfaces.go":[{"author":{"_account_id":24580,"name":"James Gu","email":"james.gu@microsoft.com","username":"jamesgu"},"change_message_id":"0c0babf6394a14cad54f65dcce0cd207b658b5c6","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"// Interface provides a abstraction layer built on top of kubectl libraries"},{"line_number":22,"context_line":"// to implement kubectl subcommands as kubectl apply"},{"line_number":23,"context_line":"type Interface interface {"},{"line_number":24,"context_line":"\tApply(docs []document.Document, ao *ApplyOptions) error"},{"line_number":25,"context_line":"\tApplyOptions() (*ApplyOptions, error)"},{"line_number":26,"context_line":"}"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_d477cd01","line":23,"updated":"2020-05-14 15:21:33.000000000","message":"IMHO, the type name \"Interface\" is too generic and confusing. But not relevant to the patch here. Maybe a TODO?","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"},{"author":{"_account_id":28690,"name":"Sirajudeen","email":"sirajudeen.yasin@gmail.com","username":"siraj.yasin"},"change_message_id":"9a2c6477f6d976dd0601c11539c8cbddd1997338","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"// Interface provides a abstraction layer built on top of kubectl libraries"},{"line_number":22,"context_line":"// to implement kubectl subcommands as kubectl apply"},{"line_number":23,"context_line":"type Interface interface {"},{"line_number":24,"context_line":"\tApply(docs []document.Document, ao *ApplyOptions) error"},{"line_number":25,"context_line":"\tApplyOptions() (*ApplyOptions, error)"},{"line_number":26,"context_line":"}"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_8c1709ee","line":23,"in_reply_to":"ff570b3c_d477cd01","updated":"2020-05-14 17:11:08.000000000","message":"Done","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"}],"testutil/testconfig.go":[{"author":{"_account_id":24580,"name":"James Gu","email":"james.gu@microsoft.com","username":"jamesgu"},"change_message_id":"0c0babf6394a14cad54f65dcce0cd207b658b5c6","unresolved":false,"context_lines":[{"line_number":235,"context_line":"\treturn authinfo"},{"line_number":236,"context_line":"}"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"// DummyBootstrapInfo creates a Dummy BootstrapInfo config object for unit testing"},{"line_number":239,"context_line":"func DummyBootstrapInfo() *config.Bootstrap {"},{"line_number":240,"context_line":"\tbs :\u003d \u0026config.Bootstrap{}"},{"line_number":241,"context_line":"\tcont :\u003d config.Container{"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_f465b146","line":238,"updated":"2020-05-14 15:21:33.000000000","message":"nit pick dummy-\u003edummy","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"},{"author":{"_account_id":28690,"name":"Sirajudeen","email":"sirajudeen.yasin@gmail.com","username":"siraj.yasin"},"change_message_id":"9a2c6477f6d976dd0601c11539c8cbddd1997338","unresolved":false,"context_lines":[{"line_number":235,"context_line":"\treturn authinfo"},{"line_number":236,"context_line":"}"},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"// DummyBootstrapInfo creates a Dummy BootstrapInfo config object for unit testing"},{"line_number":239,"context_line":"func DummyBootstrapInfo() *config.Bootstrap {"},{"line_number":240,"context_line":"\tbs :\u003d \u0026config.Bootstrap{}"},{"line_number":241,"context_line":"\tcont :\u003d config.Container{"}],"source_content_type":"text/x-go","patch_set":2,"id":"ff570b3c_ecd32537","line":238,"in_reply_to":"ff570b3c_f465b146","updated":"2020-05-14 17:11:08.000000000","message":"Done","commit_id":"df0549e3f89ebcae498b76c3ed177d213b383b03"}]}