)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"9aa12c8e2c1161986fb6bb27508d8db4b10ea34b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Darrell Bishop \u003cdarrell@swiftstack.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-23 09:11:44 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix profile tests to clean up its tempdirs."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I363651046cf414e14f15affd834043aabd5427c0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da9df570_9f960cef","line":7,"updated":"2014-09-25 01:49:03.000000000","message":"Only nitpic I have is this should really explain what it does some more. Does it fix ALL profile tests... really it just fixes the clean up problem in the Test_profile_log class. The others were working. \n\nI had no idea what the extent of what I was reviewing until I looked at the patch. Maybe the commit message needs to read more like:\n\n  Fix tempdir clean up in Test_profile_log\n\n  This change cleans up of tempdirs in the Test_profile_log\n  xprofile test class.\n\nBut not going to block on it. But if you do need to throw up a new version having a more detailed commit message is useful in case we ever need to read through the git log.","commit_id":"d10462e8704e7f5efe03c4024b541a3780348615"}]}
