)]}'
{"tools/lintstack.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1caf305056d2c8feded85a345f4d2f355106e57d","unresolved":false,"context_lines":[{"line_number":159,"context_line":"    for err_key, err_list in errors.items():"},{"line_number":160,"context_line":"        for err in err_list:"},{"line_number":161,"context_line":"            if err_key not in known:"},{"line_number":162,"context_line":"                #print err.review_str()"},{"line_number":163,"context_line":"                print err.lintoutput"},{"line_number":164,"context_line":"                print"},{"line_number":165,"context_line":"                passed \u003d False"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAALH%2F%2FoPE%3D","line":162,"updated":"2012-08-24 17:25:05.000000000","message":"femtonit: I prefer we remove code as opposed to commenting it out…","commit_id":"05afa35658c652c40c16186bfe2a556a3e4cdd24"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1caf305056d2c8feded85a345f4d2f355106e57d","unresolved":false,"context_lines":[{"line_number":170,"context_line":"            print \"Extra credit: some known pylint exceptions disappeared.\""},{"line_number":171,"context_line":"            for i in sorted(redundant):"},{"line_number":172,"context_line":"                print json.dumps(i)"},{"line_number":173,"context_line":"            print \"Consider to regenerate the exception file if you will.\""},{"line_number":174,"context_line":"    else:"},{"line_number":175,"context_line":"        print (\"Please fix the errors above. If you believe they are false\""},{"line_number":176,"context_line":"              \" positives, run \u0027tools/lintstack.py generate\u0027 to overwrite.\")"}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAALH%2F%2FoO8%3D","line":173,"updated":"2012-08-24 17:25:05.000000000","message":"femtonit: s/to regenerate/regenerating/","commit_id":"05afa35658c652c40c16186bfe2a556a3e4cdd24"}],"tools/pip-requires":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1caf305056d2c8feded85a345f4d2f355106e57d","unresolved":false,"context_lines":[{"line_number":23,"context_line":"setuptools_git\u003e\u003d0.4"},{"line_number":24,"context_line":"python-quantumclient\u003e\u003d2.0"},{"line_number":25,"context_line":"python-glanceclient\u003c2"},{"line_number":26,"context_line":"pylint\u003d\u003d0.25.2"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"AAAALH%2F%2FoOc%3D","line":26,"updated":"2012-08-24 17:25:05.000000000","message":"pylint isn\u0027t required to install nova.  I feel like this requirement should go in test-requires rather than pip-requires…","commit_id":"05afa35658c652c40c16186bfe2a556a3e4cdd24"}],"tox.ini":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"e802a3b76b27cdd0d3bfe1ca9943176d0313802a","unresolved":false,"context_lines":[{"line_number":29,"context_line":"[testenv:pylint]"},{"line_number":30,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":31,"context_line":"deps \u003d -r{toxinidir}/tools/pip-requires"},{"line_number":32,"context_line":"       pylint\u003d\u003d0.25.2"},{"line_number":33,"context_line":"commands \u003d bash tools/lintstack.sh"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[testenv:cover]"}],"source_content_type":"text/x-properties","patch_set":12,"id":"AAAALH%2F%2FnHE%3D","line":32,"updated":"2012-08-27 18:17:41.000000000","message":"Why is pylint listed as a dep here and in test-requires?","commit_id":"09eb54c48f97e7a7b9118f54c00a78fa1cbcb61d"},{"author":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"change_message_id":"b910a3233a629a8fae24aa279eeee03458f4fa45","unresolved":false,"context_lines":[{"line_number":29,"context_line":"[testenv:pylint]"},{"line_number":30,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":31,"context_line":"deps \u003d -r{toxinidir}/tools/pip-requires"},{"line_number":32,"context_line":"       pylint\u003d\u003d0.25.2"},{"line_number":33,"context_line":"commands \u003d bash tools/lintstack.sh"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[testenv:cover]"}],"source_content_type":"text/x-properties","patch_set":12,"id":"AAAALH%2F%2FnGo%3D","line":32,"in_reply_to":"AAAALH%2F%2FnHE%3D","updated":"2012-08-27 18:22:11.000000000","message":"It\u0027s in here because we don\u0027t have to install other packages in test-requires. It\u0027s in test-requires for developers to run locally. pep8 is doing the similar trick. :)","commit_id":"09eb54c48f97e7a7b9118f54c00a78fa1cbcb61d"}]}