)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"change_message_id":"a02d653708726b6ee15c496eb628bf2496371236","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2017-02-16 16:20:48 -0500"},{"line_number":4,"context_line":"Commit:     Nick Miethe \u003cnimiethe@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-16 16:20:48 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Places where we manually make lists of args and manually pass"},{"line_number":8,"context_line":"them to ringbuilder.main and expect them to return 0 should *definitely*"},{"line_number":9,"context_line":"be rewritten to use run_srb."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_fae61543","line":6,"updated":"2017-02-16 22:25:29.000000000","message":"Please insert a line that will serve as the title.  Such as \"test.unit.cli.test_ringbuilder should consisently use run_srb\"\n\nThis should be less than 70 characters, or something like that.  Has to have a blank line after it.","commit_id":"9138c304090facbce3d21566fbc57a754cfc5dbe"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"2eb3739c3d270b87cd445436f887b0cba63f0ef0","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2017-02-16 16:20:48 -0500"},{"line_number":4,"context_line":"Commit:     Nick Miethe \u003cnimiethe@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-16 16:20:48 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Places where we manually make lists of args and manually pass"},{"line_number":8,"context_line":"them to ringbuilder.main and expect them to return 0 should *definitely*"},{"line_number":9,"context_line":"be rewritten to use run_srb."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_daf4f188","line":6,"in_reply_to":"fa31d9ce_fae61543","updated":"2017-02-16 22:31:45.000000000","message":"Here is the link to the openstack commit message guidelines. It\u0027s something we all fail at times, so I\u0027ve got it bookmarked ;)\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"9138c304090facbce3d21566fbc57a754cfc5dbe"},{"author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"change_message_id":"a02d653708726b6ee15c496eb628bf2496371236","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Replaces argv and assertSystemExit(...ringbuilder.main...) with assertEqual(EXIT_..., self.run_srb(...)[0])"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This reduces code reuse and lets run_srb do the heavy lifting as intended."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I43a356efa013942942142966beecc3034ee4f6c9"},{"line_number":16,"context_line":"Partial-Bug: #1656440"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_da6eb195","line":13,"updated":"2017-02-16 22:25:29.000000000","message":"\"reduces code reuse\" sounds like a bad thing.  In this case, I don\u0027t think this last sentence is necessary because the bug itself addresses why the change should be made.","commit_id":"9138c304090facbce3d21566fbc57a754cfc5dbe"}]}