)]}'
{"bandit/bandit.py":[{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"1bf8708df0eca384cbef67180d600c614a980ebf","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    )"},{"line_number":98,"context_line":"    parser.add_argument("},{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results filter, show only issue of this level or \u0027"},{"line_number":101,"context_line":"                         \u0027higher. -l for LOW -ll for MEDIUM -lll for HIGH\u0027)"},{"line_number":102,"context_line":"    )"},{"line_number":103,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_c53e698a","line":100,"updated":"2015-07-29 15:17:15.000000000","message":"s/issue/issues","commit_id":"dcfae77952f3cf37395cbe7737cc38f476004599"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"a6ef16df2b19c804db185a9fd9d6917c5eb184da","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    )"},{"line_number":98,"context_line":"    parser.add_argument("},{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results filter, show only issues of this level or \u0027"},{"line_number":101,"context_line":"                         \u0027higher. -l for LOW -ll for MEDIUM -lll for HIGH\u0027)"},{"line_number":102,"context_line":"    )"},{"line_number":103,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_171b1574","line":100,"updated":"2015-07-29 20:17:43.000000000","message":"Actually this should state this is a level of severity, not confidence.","commit_id":"42e958ad2489472ebc64ef113b9c3f42c66cd298"},{"author":{"_account_id":11716,"name":"Tim Kelsey","email":"timothy.kelsey@uk.ibm.com","username":"tkelsey"},"change_message_id":"34a67e1e606edca34ff2505b26a16c01c879ffda","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    )"},{"line_number":98,"context_line":"    parser.add_argument("},{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results filter, show only issues of this level or \u0027"},{"line_number":101,"context_line":"                         \u0027higher. -l for LOW -ll for MEDIUM -lll for HIGH\u0027)"},{"line_number":102,"context_line":"    )"},{"line_number":103,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_24f6fdf9","line":100,"in_reply_to":"3a50d1a3_171b1574","updated":"2015-07-30 12:40:36.000000000","message":"Yes, good point Eric, will do.","commit_id":"42e958ad2489472ebc64ef113b9c3f42c66cd298"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"a2b96aefdf895c0ae3f3fef720becfe57737014c","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    parser.add_argument("},{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results filter, show only issues of this level or \u0027"},{"line_number":101,"context_line":"                         \u0027higher. -l for LOW -ll for MEDIUM -lll for HIGH\u0027)"},{"line_number":102,"context_line":"    )"},{"line_number":103,"context_line":"    parser.add_argument("},{"line_number":104,"context_line":"        \u0027-f\u0027, \u0027--format\u0027, dest\u003d\u0027output_format\u0027, action\u003d\u0027store\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_3e65da43","line":101,"updated":"2015-07-29 16:19:45.000000000","message":"commas between the options would be nice... e.g. \"Use -l for LOW, -ll for...\"\n\nWhy is the default not one of the expected values? Either the help is missing another option, or the default should be one should be one of the options that the help lists.\n\nIt would also be nice to see this using argparse\u0027s choices support. See https://docs.python.org/2/library/argparse.html#choices","commit_id":"42e958ad2489472ebc64ef113b9c3f42c66cd298"},{"author":{"_account_id":11716,"name":"Tim Kelsey","email":"timothy.kelsey@uk.ibm.com","username":"tkelsey"},"change_message_id":"34a67e1e606edca34ff2505b26a16c01c879ffda","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    parser.add_argument("},{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results filter, show only issues of this level or \u0027"},{"line_number":101,"context_line":"                         \u0027higher. -l for LOW -ll for MEDIUM -lll for HIGH\u0027)"},{"line_number":102,"context_line":"    )"},{"line_number":103,"context_line":"    parser.add_argument("},{"line_number":104,"context_line":"        \u0027-f\u0027, \u0027--format\u0027, dest\u003d\u0027output_format\u0027, action\u003d\u0027store\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_c494f98d","line":101,"in_reply_to":"3a50d1a3_3e65da43","updated":"2015-07-30 12:40:36.000000000","message":"It looks like choices would have been a better choice here, however I don\u0027t want to change the CLI API at this point. I\u0027ll add the commas as you suggest. The default value is one of the options, just a single \"-l\" I\u0027m not sure what you mean here?","commit_id":"42e958ad2489472ebc64ef113b9c3f42c66cd298"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"c05b98df332b679f1f2803163ee6df4c41c4d4e0","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results severity filter. Show only issues of a given\u0027"},{"line_number":101,"context_line":"                         \u0027 severity level or higher. -l for LOW,\u0027"},{"line_number":102,"context_line":"                         \u0027 -ll for MEDIUM, -lll for HIGH\u0027)"},{"line_number":103,"context_line":"    )"},{"line_number":104,"context_line":"    parser.add_argument("},{"line_number":105,"context_line":"        \u0027-f\u0027, \u0027--format\u0027, dest\u003d\u0027output_format\u0027, action\u003d\u0027store\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_c724e2b3","line":102,"updated":"2015-07-30 22:23:02.000000000","message":"why not take the level name as a string, then the docs wouldn\u0027t be needed.\n\nAlso, since this is a severity filter how about make it --severity instead of --level?","commit_id":"86552d044f3ad6153c4ce2e945a7c971747d8fcb"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"8e63c8d6a83249fbed31b9e4d8a142be4456b04e","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        \u0027-l\u0027, \u0027--level\u0027, dest\u003d\u0027level\u0027, action\u003d\u0027count\u0027,"},{"line_number":100,"context_line":"        default\u003d1, help\u003d(\u0027results severity filter. Show only issues of a given\u0027"},{"line_number":101,"context_line":"                         \u0027 severity level or higher. -l for LOW,\u0027"},{"line_number":102,"context_line":"                         \u0027 -ll for MEDIUM, -lll for HIGH\u0027)"},{"line_number":103,"context_line":"    )"},{"line_number":104,"context_line":"    parser.add_argument("},{"line_number":105,"context_line":"        \u0027-f\u0027, \u0027--format\u0027, dest\u003d\u0027output_format\u0027, action\u003d\u0027store\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_27e34635","line":102,"in_reply_to":"3a50d1a3_c724e2b3","updated":"2015-07-30 22:32:17.000000000","message":"Although I agree it would be nice to change, at this point it would potentially break projects already using \u0027-l\u0027 (like Nova).  Unless of course an alias can be made so that both the old and new parameter can be honored.","commit_id":"86552d044f3ad6153c4ce2e945a7c971747d8fcb"}]}