)]}'
{"app/js/on_config.js":[{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"88f8adda2e053b8fd179304ab55fdcd03eed39a2","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      title: \u0027Tests\u0027"},{"line_number":33,"context_line":"    })"},{"line_number":34,"context_line":"    .state(\u0027testsDetail\u0027, {"},{"line_number":35,"context_line":"      url: \u0027/testsDetail/:key\u0027,"},{"line_number":36,"context_line":"      controller: \u0027TestsDetailController as testsDetail\u0027,"},{"line_number":37,"context_line":"      templateUrl: \u0027tests-detail.html\u0027,"},{"line_number":38,"context_line":"      title: \u0027Tests Detail\u0027,"}],"source_content_type":"text/javascript","patch_set":1,"id":"bae84128_5a2eb243","line":35,"range":{"start_line":35,"start_character":13,"end_line":35,"end_character":24},"updated":"2016-02-22 23:00:45.000000000","message":"I\u0027d recommend just /tests/:key here","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2560929ab3971a026410a6cd2d4d1eb695dc4055","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      title: \u0027Tests\u0027"},{"line_number":33,"context_line":"    })"},{"line_number":34,"context_line":"    .state(\u0027testsDetail\u0027, {"},{"line_number":35,"context_line":"      url: \u0027/testsDetail/:key\u0027,"},{"line_number":36,"context_line":"      controller: \u0027TestsDetailController as testsDetail\u0027,"},{"line_number":37,"context_line":"      templateUrl: \u0027tests-detail.html\u0027,"},{"line_number":38,"context_line":"      title: \u0027Tests Detail\u0027,"}],"source_content_type":"text/javascript","patch_set":1,"id":"bae84128_e77eed68","line":35,"range":{"start_line":35,"start_character":13,"end_line":35,"end_character":24},"in_reply_to":"bae84128_5a2eb243","updated":"2016-02-23 17:04:01.000000000","message":"Done","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"}],"app/views/tests.html":[{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"88f8adda2e053b8fd179304ab55fdcd03eed39a2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    \u003c/div\u003e"},{"line_number":14,"context_line":"    \u003cdiv class\u003d\"row\"\u003e"},{"line_number":15,"context_line":"        \u003cdiv class\u003d\"col-lg-12\"\u003e"},{"line_number":16,"context_line":"          \u003cdiv class\u003d\"panel panel-default\"\u003e"},{"line_number":17,"context_line":"            \u003cdiv class\u003d\"panel-heading\"\u003e"},{"line_number":18,"context_line":"                \u003ch3 class\u003d\"panel-title\"\u003eDetails\u003c/h3\u003e"},{"line_number":19,"context_line":"            \u003c/div\u003e"}],"source_content_type":"text/html","patch_set":1,"id":"bae84128_bac9be5b","line":16,"range":{"start_line":16,"start_character":10,"end_line":16,"end_character":43},"updated":"2016-02-22 23:00:45.000000000","message":"nit: there should be 4 spaces of indent per level","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2560929ab3971a026410a6cd2d4d1eb695dc4055","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    \u003c/div\u003e"},{"line_number":14,"context_line":"    \u003cdiv class\u003d\"row\"\u003e"},{"line_number":15,"context_line":"        \u003cdiv class\u003d\"col-lg-12\"\u003e"},{"line_number":16,"context_line":"          \u003cdiv class\u003d\"panel panel-default\"\u003e"},{"line_number":17,"context_line":"            \u003cdiv class\u003d\"panel-heading\"\u003e"},{"line_number":18,"context_line":"                \u003ch3 class\u003d\"panel-title\"\u003eDetails\u003c/h3\u003e"},{"line_number":19,"context_line":"            \u003c/div\u003e"}],"source_content_type":"text/html","patch_set":1,"id":"bae84128_42434742","line":16,"range":{"start_line":16,"start_character":10,"end_line":16,"end_character":43},"in_reply_to":"bae84128_bac9be5b","updated":"2016-02-23 17:04:01.000000000","message":"Done","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"},{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"88f8adda2e053b8fd179304ab55fdcd03eed39a2","unresolved":false,"context_lines":[{"line_number":18,"context_line":"                \u003ch3 class\u003d\"panel-title\"\u003eDetails\u003c/h3\u003e"},{"line_number":19,"context_line":"            \u003c/div\u003e"},{"line_number":20,"context_line":"            \u003cdiv class\u003d\"panel-body\"\u003e"},{"line_number":21,"context_line":"              \u003cul ng-repeat\u003d\"(key, value) in tests.chartData\"\u003e"},{"line_number":22,"context_line":"                    \u003cli\u003e\u003ca ui-sref\u003d\"testsDetail({ key: key })\"\u003e{{ key }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":23,"context_line":"              \u003c/ul\u003e"},{"line_number":24,"context_line":"            \u003c/div\u003e"}],"source_content_type":"text/html","patch_set":1,"id":"bae84128_fad6a6ef","line":21,"range":{"start_line":21,"start_character":18,"end_line":21,"end_character":61},"updated":"2016-02-22 23:00:45.000000000","message":"small bug: ng-repeat should be on \u003cli\u003e\n\nI\u0027d also recommend converting this to a list group, e.g. http://getbootstrap.com/components/#list-group-linked - should be a quick way to make this look a bit nicer.","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2560929ab3971a026410a6cd2d4d1eb695dc4055","unresolved":false,"context_lines":[{"line_number":18,"context_line":"                \u003ch3 class\u003d\"panel-title\"\u003eDetails\u003c/h3\u003e"},{"line_number":19,"context_line":"            \u003c/div\u003e"},{"line_number":20,"context_line":"            \u003cdiv class\u003d\"panel-body\"\u003e"},{"line_number":21,"context_line":"              \u003cul ng-repeat\u003d\"(key, value) in tests.chartData\"\u003e"},{"line_number":22,"context_line":"                    \u003cli\u003e\u003ca ui-sref\u003d\"testsDetail({ key: key })\"\u003e{{ key }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":23,"context_line":"              \u003c/ul\u003e"},{"line_number":24,"context_line":"            \u003c/div\u003e"}],"source_content_type":"text/html","patch_set":1,"id":"bae84128_2239e3d7","line":21,"range":{"start_line":21,"start_character":18,"end_line":21,"end_character":61},"in_reply_to":"bae84128_fad6a6ef","updated":"2016-02-23 17:04:01.000000000","message":"It\u0027s really nice, thanks :) Done","commit_id":"f1a594a6dca000190b26ec5d372501de15e34194"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"908a7332de5e298297c47417cdcbd08ad5552d14","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        \u003cdiv class\u003d\"col-lg-12\"\u003e"},{"line_number":16,"context_line":"            \u003cdiv class\u003d\"panel panel-default\"\u003e"},{"line_number":17,"context_line":"                \u003cdiv class\u003d\"panel-heading\"\u003e"},{"line_number":18,"context_line":"                    \u003ch3 class\u003d\"panel-title\"\u003eDetails list\u003c/h3\u003e"},{"line_number":19,"context_line":"                \u003c/div\u003e"},{"line_number":20,"context_line":"                \u003cdiv class\u003d\"panel-body\"\u003e"},{"line_number":21,"context_line":"                    \u003cdiv class\u003d\"list-group\"\u003e"}],"source_content_type":"text/html","patch_set":3,"id":"bae84128_4fae79e8","line":18,"range":{"start_line":18,"start_character":44,"end_line":18,"end_character":56},"updated":"2016-02-24 18:47:41.000000000","message":"I don\u0027t think this is very clear to users, can we push a followup change that changes this to something like:\n\nTest Suite Groups\n\nOr something like that. Just to explain that it\u0027s a list of grouped tests from the same directory. But I don\u0027t think we should block this on that detail.","commit_id":"8a0ab1d42563fc7650b1696cfe12d28bc5058b03"}]}