)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"f6e1175d820f478013a6a22e62b10af589d92c13","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Decomposition of Pathset https://review.opendev.org/#/c/693595/"},{"line_number":8,"context_line":"into multiple smaller patchsets"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Introduces required Context cmds Types, Test and Test utils."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Icdb128d499d30e04156afd84976376318c7b1ae4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_88d6ccf1","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":60},"updated":"2019-11-21 17:57:31.000000000","message":"Would you mind moving this to the top of the message? I think it better describes the intent of this commit","commit_id":"1f13ba2ce4583ed1229a21261f35e1ba14c6166b"},{"author":{"_account_id":7185,"name":"Dmitry Ukov","email":"dukov@mirantis.com","username":"dukov"},"change_message_id":"0e778c2de67156dd1e8674e300515d32bf1d51c4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rodolfo Pacheco \u003cjezogwza@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-11-21 20:16:54 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Introduces required Context cmds Types, Test and Test utils."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Decomposition of Pathset https://review.opendev.org/#/c/693595/"},{"line_number":10,"context_line":"into multiple smaller patchsets"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_8839e53f","line":7,"range":{"start_line":7,"start_character":8,"end_line":7,"end_character":10},"updated":"2019-11-22 04:23:35.000000000","message":"just nit: introduce\n\nBut this is more like a personal preference","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"},{"author":{"_account_id":22259,"name":"Nishant Kumar","email":"nishant.e.kumar@att.com","username":"nishant.e.kumar"},"change_message_id":"a51bbbef01aabb2781f860d0cc9f613884419cc4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rodolfo Pacheco \u003cjezogwza@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-11-21 20:16:54 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Introduces required Context cmds Types, Test and Test utils."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Decomposition of Pathset https://review.opendev.org/#/c/693595/"},{"line_number":10,"context_line":"into multiple smaller patchsets"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_03038c11","line":7,"range":{"start_line":7,"start_character":8,"end_line":7,"end_character":10},"in_reply_to":"3fa7e38b_8839e53f","updated":"2019-11-22 18:02:18.000000000","message":"Is this related to any story number?","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"},{"author":{"_account_id":7185,"name":"Dmitry Ukov","email":"dukov@mirantis.com","username":"dukov"},"change_message_id":"0e778c2de67156dd1e8674e300515d32bf1d51c4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Introduces required Context cmds Types, Test and Test utils."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Decomposition of Pathset https://review.opendev.org/#/c/693595/"},{"line_number":10,"context_line":"into multiple smaller patchsets"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_284471b7","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":31},"updated":"2019-11-22 04:23:35.000000000","message":"not sure that we need to specify this in commit message.","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"}],"pkg/config/cmds.go":[{"author":{"_account_id":22259,"name":"Nishant Kumar","email":"nishant.e.kumar@att.com","username":"nishant.e.kumar"},"change_message_id":"a51bbbef01aabb2781f860d0cc9f613884419cc4","unresolved":false,"context_lines":[{"line_number":48,"context_line":"\treturn nil"},{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"func (o *ContextOptions) Validate() error {"},{"line_number":52,"context_line":"\tif len(o.Name) \u003d\u003d 0 {"},{"line_number":53,"context_line":"\t\treturn errors.New(\"you must specify a non-empty context name\")"},{"line_number":54,"context_line":"\t}"}],"source_content_type":"text/x-go","patch_set":2,"id":"3fa7e38b_63acc034","line":51,"range":{"start_line":51,"start_character":6,"end_line":51,"end_character":7},"updated":"2019-11-22 18:02:18.000000000","message":"nit: could be more meaningful.","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"0b997dcf2317fd0f40b042d52e652ce9a36fb433","unresolved":false,"context_lines":[{"line_number":48,"context_line":"\treturn nil"},{"line_number":49,"context_line":"}"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"func (o *ContextOptions) Validate() error {"},{"line_number":52,"context_line":"\tif len(o.Name) \u003d\u003d 0 {"},{"line_number":53,"context_line":"\t\treturn errors.New(\"you must specify a non-empty context name\")"},{"line_number":54,"context_line":"\t}"}],"source_content_type":"text/x-go","patch_set":2,"id":"3fa7e38b_07cf027f","line":51,"range":{"start_line":51,"start_character":6,"end_line":51,"end_character":7},"in_reply_to":"3fa7e38b_63acc034","updated":"2019-11-25 17:56:20.000000000","message":"One to two characters is standard Golang - https://github.com/golang/go/wiki/CodeReviewComments#receiver-names","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"}],"pkg/config/cmds_test.go":[{"author":{"_account_id":22259,"name":"Nishant Kumar","email":"nishant.e.kumar@att.com","username":"nishant.e.kumar"},"change_message_id":"a51bbbef01aabb2781f860d0cc9f613884419cc4","unresolved":false,"context_lines":[{"line_number":72,"context_line":"\tco.ClusterType \u003d \"Invalid\""},{"line_number":73,"context_line":"\terr \u003d co.Validate()"},{"line_number":74,"context_line":"\tassert.Error(t, err)"},{"line_number":75,"context_line":"\t// Empty Name"},{"line_number":76,"context_line":"\tco.ClusterType \u003d Target"},{"line_number":77,"context_line":"\tco.Name \u003d \"\""},{"line_number":78,"context_line":"\terr \u003d co.Validate()"}],"source_content_type":"text/x-go","patch_set":2,"id":"3fa7e38b_a33958f1","line":75,"range":{"start_line":75,"start_character":1,"end_line":75,"end_character":14},"updated":"2019-11-22 18:02:18.000000000","message":"nit: Line spacing missed, this can go L76","commit_id":"c047cdca404e57998b53a371fafd7959ae6f7fcb"}]}