)]}'
{"manifests/packages.pp":[{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"05550f94e7cce10f975f7813106ed3199452cb9a","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  $enable_upgrade \u003d false,"},{"line_number":33,"context_line":") {"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"  if !$enable_install and !$enable_update {"},{"line_number":36,"context_line":"    case $::osfamily {"},{"line_number":37,"context_line":"      \u0027RedHat\u0027: {"},{"line_number":38,"context_line":"        Package \u003c| |\u003e { provider \u003d\u003e \u0027norpm\u0027 }"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9a1a9d01_1c48a32c","line":35,"updated":"2015-09-28 16:33:09.000000000","message":"The double negative here is a bit confusing. Perhaps naming the parameters disable_install and disable_upgrade would have made this read more cleanly? Would require deprecation, etc to change that now though. Probably best to leave as is and potentially refactor later.","commit_id":"ebe291862bb6adbce4372f8c8524a346ff1859c3"}]}