)]}'
{"pbr/tests/base.py":[{"author":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"change_message_id":"56ba0a860ab36f03d25533c1c09f6e024d7d7dfc","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"def _config_git():"},{"line_number":167,"context_line":"    _run_cmd("},{"line_number":168,"context_line":"        [\u0027git\u0027, \u0027config\u0027, \u0027--global\u0027, \u0027user.email\u0027, \u0027example@example.com\u0027],"},{"line_number":169,"context_line":"        None)"},{"line_number":170,"context_line":"    _run_cmd("},{"line_number":171,"context_line":"        [\u0027git\u0027, \u0027config\u0027, \u0027--global\u0027, \u0027user.name\u0027, \u0027OpenStack Developer\u0027],"}],"source_content_type":"text/x-python","patch_set":13,"id":"3acd31a7_56bbace5","line":168,"updated":"2015-05-01 04:11:28.000000000","message":"(nit) I would have just written this out with .split() but that\u0027s a matter of opinion","commit_id":"58daa9f2b95e56f129d112245d1b700adec7a08a"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"35a68ed98a4b171588321eff9cc9f95120a1c5f5","unresolved":false,"context_lines":[{"line_number":165,"context_line":""},{"line_number":166,"context_line":"def _config_git():"},{"line_number":167,"context_line":"    _run_cmd("},{"line_number":168,"context_line":"        [\u0027git\u0027, \u0027config\u0027, \u0027--global\u0027, \u0027user.email\u0027, \u0027example@example.com\u0027],"},{"line_number":169,"context_line":"        None)"},{"line_number":170,"context_line":"    _run_cmd("},{"line_number":171,"context_line":"        [\u0027git\u0027, \u0027config\u0027, \u0027--global\u0027, \u0027user.name\u0027, \u0027OpenStack Developer\u0027],"}],"source_content_type":"text/x-python","patch_set":13,"id":"3acd31a7_794578b1","line":168,"in_reply_to":"3acd31a7_56bbace5","updated":"2015-05-01 12:27:36.000000000","message":"The security dork in me prefers to see command lines parameterized even when they\u0027re entirely hard-coded, just because it\u0027s easier to forget/overlook a missing split(), and easier not to notice when there\u0027s a non-format-string variable injected in the mix.","commit_id":"58daa9f2b95e56f129d112245d1b700adec7a08a"}]}