)]}'
{"etc/os_loganalyze/wsgi.conf":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"022b778b9fc1ca9753d1d551c64c59acdd6a97ef","unresolved":false,"context_lines":[{"line_number":3,"context_line":"view \u003d HTMLView"},{"line_number":4,"context_line":"file_conditions \u003d /etc/os-loganalyze/file_conditions.yaml"},{"line_number":5,"context_line":"generate_folder_index \u003d true"},{"line_number":6,"context_line":"colorize \u003d false"}],"source_content_type":"text/plain","patch_set":11,"id":"9f6a8fd7_78696289","line":6,"updated":"2018-04-30 13:20:10.000000000","message":"It would be nice if (at least the public-facing versions of) this keyword - the conf and the queryparam - indicated the ansi-ness of the color scheme.  E.g. ansi_colorize.  Because I would eventually like to see other color scheme possibilities, à la [1]\n\n[1] https://review.openstack.org/#/c/524744/","commit_id":"9587594d4c374f9267c45fe25e9a4b6692e2e2ec"}],"os_loganalyze/view.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"022b778b9fc1ca9753d1d551c64c59acdd6a97ef","unresolved":false,"context_lines":[{"line_number":154,"context_line":"NO_ESCAPE_FINISH \u003d re.compile(\"\u003c/pre\u003e\")"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"# stop: 0"},{"line_number":157,"context_line":"ANSICOLORFMT \u003d re.compile(\u0027\\x1b\\[(?P\u003cfg_color\u003e\\d+)(;(?P\u003cbg_color\u003e\\d+))?m\u0027)"},{"line_number":158,"context_line":"ANSICOLOREIGHTBITFMT \u003d re.compile("},{"line_number":159,"context_line":"    \u0027\\x1b\\[(?P\u003cfb_selector\u003e38|48)[:;]5[:;](?P\u003crgb_color\u003e\\d+)m\u0027)"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"9f6a8fd7_b8aa9acf","line":157,"range":{"start_line":157,"start_character":37,"end_line":157,"end_character":45},"updated":"2018-04-30 13:20:10.000000000","message":"Just a note, you don\u0027t seem to be using these tags anywhere.  But they\u0027re not hurting anything, and they\u0027re nice for readability.","commit_id":"9587594d4c374f9267c45fe25e9a4b6692e2e2ec"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"022b778b9fc1ca9753d1d551c64c59acdd6a97ef","unresolved":false,"context_lines":[{"line_number":154,"context_line":"NO_ESCAPE_FINISH \u003d re.compile(\"\u003c/pre\u003e\")"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"# stop: 0"},{"line_number":157,"context_line":"ANSICOLORFMT \u003d re.compile(\u0027\\x1b\\[(?P\u003cfg_color\u003e\\d+)(;(?P\u003cbg_color\u003e\\d+))?m\u0027)"},{"line_number":158,"context_line":"ANSICOLOREIGHTBITFMT \u003d re.compile("},{"line_number":159,"context_line":"    \u0027\\x1b\\[(?P\u003cfb_selector\u003e38|48)[:;]5[:;](?P\u003crgb_color\u003e\\d+)m\u0027)"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"9f6a8fd7_f87332b0","line":157,"range":{"start_line":157,"start_character":50,"end_line":157,"end_character":51},"updated":"2018-04-30 13:20:10.000000000","message":"You can use (?: here to make this non-capturing (so you don\u0027t need the _ on L206).","commit_id":"9587594d4c374f9267c45fe25e9a4b6692e2e2ec"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"022b778b9fc1ca9753d1d551c64c59acdd6a97ef","unresolved":false,"context_lines":[{"line_number":412,"context_line":""},{"line_number":413,"context_line":"    # Do not allow log coloring if disabled in the configuration"},{"line_number":414,"context_line":"    if not (config.has_section(\u0027general\u0027) and"},{"line_number":415,"context_line":"       config.has_option(\u0027general\u0027, \u0027colorize\u0027) and"},{"line_number":416,"context_line":"       config.getboolean(\u0027general\u0027, \u0027colorize\u0027)):"},{"line_number":417,"context_line":"        colorize \u003d False"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":"    if view_selected:"}],"source_content_type":"text/x-python","patch_set":11,"id":"9f6a8fd7_f881f24b","line":416,"range":{"start_line":415,"start_character":0,"end_line":416,"end_character":49},"updated":"2018-04-30 13:20:10.000000000","message":"Consider double-indenting these for readability.","commit_id":"9587594d4c374f9267c45fe25e9a4b6692e2e2ec"}]}