)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"27bf0acd31588c614f0a45dcd471c925f7961ac2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add some discussion about review criteria"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"An attempt to layout some of the ratioanle behind devstack reviews."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I9f4878653b5c746159206cd44b49255d9fdd32ef"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a6ced46_8dcc8e37","line":9,"updated":"2015-03-23 12:52:10.000000000","message":"two words for lay out (layout is something else) and \"rationale\"","commit_id":"c7df4df0b18a50313497bfca31af04e5475f780f"}],"HACKING.rst":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"a2a0cd27799bf7e2e8994b6a59f71c346f390d8d","unresolved":false,"context_lines":[{"line_number":331,"context_line":"* **Is it passing tests** -- your change will not be reviewed"},{"line_number":332,"context_line":"  throughly unless the official CI has run successfully against it."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"* **Does this belong in devstack** -- devstack reviewers have a"},{"line_number":335,"context_line":"  default position of \"no\" but are ready to be convinced by your"},{"line_number":336,"context_line":"  change."},{"line_number":337,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a80dd14_7d1ca813","line":334,"updated":"2015-03-20 14:21:20.000000000","message":"please capitalize DevStack camel-case when referring to the project.","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"66a06023f90ede03b6fd0cd962bf9e5ace721804","unresolved":false,"context_lines":[{"line_number":331,"context_line":"* **Is it passing tests** -- your change will not be reviewed"},{"line_number":332,"context_line":"  throughly unless the official CI has run successfully against it."},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"* **Does this belong in devstack** -- devstack reviewers have a"},{"line_number":335,"context_line":"  default position of \"no\" but are ready to be convinced by your"},{"line_number":336,"context_line":"  change."},{"line_number":337,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6ced46_600a0bd4","line":334,"in_reply_to":"9a80dd14_7d1ca813","updated":"2015-03-23 03:07:21.000000000","message":"Done","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"a2a0cd27799bf7e2e8994b6a59f71c346f390d8d","unresolved":false,"context_lines":[{"line_number":340,"context_line":"  repository."},{"line_number":341,"context_line":""},{"line_number":342,"context_line":"  For smaller changes, you should always consider if the change can be"},{"line_number":343,"context_line":"  encapsulated by per-user settings in ``localrc``.  A common example"},{"line_number":344,"context_line":"  is adding a simple config-option to an ``ini`` file.  Specific flags"},{"line_number":345,"context_line":"  are not usually required for this, although adding documentation"},{"line_number":346,"context_line":"  about how to achieve a larger goal (which might include turning on"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a80dd14_3d42300f","line":343,"updated":"2015-03-20 14:21:20.000000000","message":"``local.conf``","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"66a06023f90ede03b6fd0cd962bf9e5ace721804","unresolved":false,"context_lines":[{"line_number":340,"context_line":"  repository."},{"line_number":341,"context_line":""},{"line_number":342,"context_line":"  For smaller changes, you should always consider if the change can be"},{"line_number":343,"context_line":"  encapsulated by per-user settings in ``localrc``.  A common example"},{"line_number":344,"context_line":"  is adding a simple config-option to an ``ini`` file.  Specific flags"},{"line_number":345,"context_line":"  are not usually required for this, although adding documentation"},{"line_number":346,"context_line":"  about how to achieve a larger goal (which might include turning on"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6ced46_800f07e1","line":343,"in_reply_to":"9a80dd14_3d42300f","updated":"2015-03-23 03:07:21.000000000","message":"Done","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"a2a0cd27799bf7e2e8994b6a59f71c346f390d8d","unresolved":false,"context_lines":[{"line_number":351,"context_line":"  presented in the context of fixing the root-cause of the problem."},{"line_number":352,"context_line":"  This means it is well-commented in the code and the change-log and"},{"line_number":353,"context_line":"  mostly likely includes links to changes or bugs that fix the"},{"line_number":354,"context_line":"  underlying problem."},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"* **Should this be upstream** -- devstack generally does not override"},{"line_number":357,"context_line":"  default choices provided by projects and attempts to not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a80dd14_5d73647f","line":354,"updated":"2015-03-20 14:21:20.000000000","message":"System-level workarounds should also try to go into ``tools/fixup_stuff.sh`` first to keep them together.","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"66a06023f90ede03b6fd0cd962bf9e5ace721804","unresolved":false,"context_lines":[{"line_number":351,"context_line":"  presented in the context of fixing the root-cause of the problem."},{"line_number":352,"context_line":"  This means it is well-commented in the code and the change-log and"},{"line_number":353,"context_line":"  mostly likely includes links to changes or bugs that fix the"},{"line_number":354,"context_line":"  underlying problem."},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"* **Should this be upstream** -- devstack generally does not override"},{"line_number":357,"context_line":"  default choices provided by projects and attempts to not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6ced46_402c2f4b","line":354,"in_reply_to":"9a80dd14_5d73647f","updated":"2015-03-23 03:07:21.000000000","message":"Done","commit_id":"e713c03e6b8b9b5f2395d5de5693473c4594f27e"}]}