)]}'
{"modules/gerrit/manifests/init.pp":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"unresolved":false,"context_lines":[{"line_number":367,"context_line":"      owner   \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":368,"context_line":"      group   \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":369,"context_line":"      mode    \u003d\u003e \u00270644\u0027,"},{"line_number":370,"context_line":"      content \u003d\u003e $ssh_project_rsa_pubkey_contents,"},{"line_number":371,"context_line":"      replace \u003d\u003e true,"},{"line_number":372,"context_line":"      require \u003d\u003e File[\u0027/home/gerrit2/review_site/etc\u0027]"},{"line_number":373,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"AAAAMH%2F%2Fz38%3D","line":370,"updated":"2012-11-21 20:53:17.000000000","message":"this looks like a bug fix?","commit_id":"9d0c1d8cc72a2250ab31634568d9d7f0e06507d6"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1a3f52c8170283bbd298822ed861424621de946e","unresolved":false,"context_lines":[{"line_number":367,"context_line":"      owner   \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":368,"context_line":"      group   \u003d\u003e \u0027gerrit2\u0027,"},{"line_number":369,"context_line":"      mode    \u003d\u003e \u00270644\u0027,"},{"line_number":370,"context_line":"      content \u003d\u003e $ssh_project_rsa_pubkey_contents,"},{"line_number":371,"context_line":"      replace \u003d\u003e true,"},{"line_number":372,"context_line":"      require \u003d\u003e File[\u0027/home/gerrit2/review_site/etc\u0027]"},{"line_number":373,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"AAAAMH%2F%2Fzzg%3D","line":370,"updated":"2012-11-21 21:51:15.000000000","message":"Bugfixes should normally be separated from code cleanup, but that file doesn\u0027t actually seem to be actively used by the module (wasn\u0027t breaking the feature) so I think it\u0027s probably okay for this patchset. If you wind up doing another iteration, though, it might be better moved to a separate commit.","commit_id":"3a9c3f041419340e22006a7c30d0ba19d863acac"}],"modules/jenkins/manifests/master.pp":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"1a3f52c8170283bbd298822ed861424621de946e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# \u003d\u003d Class: jenkins::master"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"class jenkins::master("},{"line_number":4,"context_line":"  $logo \u0027\u0027,"},{"line_number":5,"context_line":"  $vhost_name \u003d $::fqdn,"},{"line_number":6,"context_line":"  $serveradmin \u003d \"webmaster@${::fqdn}\","},{"line_number":7,"context_line":"  $ssl_cert_file \u003d \u0027\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"AAAAMH%2F%2Fz7Q%3D","line":4,"updated":"2012-11-21 20:14:19.000000000","message":"operator?","commit_id":"9d0c1d8cc72a2250ab31634568d9d7f0e06507d6"}],"modules/unattended_upgrades/manifests/init.pp":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"unresolved":false,"context_lines":[{"line_number":1,"context_line":"# \u003d\u003d Class: unattended_upgrades"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"class unattended_upgrades($ensure \u003d present) {"},{"line_number":4,"context_line":"  package { \u0027unattended-upgrades\u0027:"},{"line_number":5,"context_line":"    ensure \u003d\u003e $ensure,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"AAAAMH%2F%2Fz3Y%3D","line":2,"updated":"2012-11-21 20:53:17.000000000","message":"Missing #","commit_id":"9d0c1d8cc72a2250ab31634568d9d7f0e06507d6"}]}