)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"0037526f5731f122734e600888e6320024b2a5cf","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Joachim Barheine \u003cjoachim.barheine@sap.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-27 12:51:34 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"introduce --skip_detection_plugins option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I813b8e3aba1277b321a11db97ce2aef6a9e960b4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9abb7d3a_07057cc1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":9},"updated":"2016-05-27 12:43:52.000000000","message":"should this be something more like add skip detection plugin option that allows skipping a list of specified plugins when running monasca setup.","commit_id":"e3e9feab019d4ab6c294675874a7d7e6bc6d3bfd"}],"monasca_setup/utils.py":[{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"0037526f5731f122734e600888e6320024b2a5cf","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    lower_plugins \u003d [p.lower() for p in plugin_names]"},{"line_number":55,"context_line":"    plugins \u003d []"},{"line_number":56,"context_line":"    for plugin in plugin_list:"},{"line_number":57,"context_line":"        b \u003d (plugin.__name__.lower() in lower_plugins)"},{"line_number":58,"context_line":"        if b !\u003d skip:"},{"line_number":59,"context_line":"            plugins.append(plugin)"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_27cfe0b1","line":57,"range":{"start_line":57,"start_character":8,"end_line":57,"end_character":9},"updated":"2016-05-27 12:43:52.000000000","message":"instead of using b as a local variable can we use plugin_status or plugin_exists? and could make this one conditional like \n\nif plugin.__name__.lower() in lower_plugins and skip\n\nJust seems cleaner to me instead of b","commit_id":"e3e9feab019d4ab6c294675874a7d7e6bc6d3bfd"}]}