)]}'
{"tox.ini":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"25f5855df6f7656a73664973faea44465ad4ecb8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"       ."},{"line_number":33,"context_line":"commands \u003d flake8 {posargs} bandit"},{"line_number":34,"context_line":"           flake8 {posargs} tests"},{"line_number":35,"context_line":"           bandit-baseline -r bandit -ll -ii"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"[testenv:venv]"},{"line_number":38,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7a77a97e_3d2a57e8","line":35,"updated":"2016-11-21 19:06:12.000000000","message":"I hate megapatches. They\u0027re impossible to effectively review. I would, in my opinion, tweak the pylintrc to get it to pass then incrementally remove things so that it passes each time.","commit_id":"bd621bf49cdc6e672704bac0ddfe17ae91c18ac5"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"908678f79a26e570c080519bf1d8a1094032c01a","unresolved":false,"context_lines":[{"line_number":32,"context_line":"       ."},{"line_number":33,"context_line":"commands \u003d flake8 {posargs} bandit"},{"line_number":34,"context_line":"           flake8 {posargs} tests"},{"line_number":35,"context_line":"           bandit-baseline -r bandit -ll -ii"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"[testenv:venv]"},{"line_number":38,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7a77a97e_dd0fa32c","line":35,"updated":"2016-11-21 18:49:52.000000000","message":"We should also run the pylint command here so that it\u0027s auto-included in our pep8 job and we don\u0027t need a new linter job. That allows it to be removed from the envlist above.\n\nFor example,\n\n    {[testenv:pylint]commands}\n\nAlso doesn\u0027t this necessitate adding pylint to our test-requirements file?","commit_id":"bd621bf49cdc6e672704bac0ddfe17ae91c18ac5"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"5f8af7c6c21af92c62df89a0f3e286408b99c6c8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"       ."},{"line_number":33,"context_line":"commands \u003d flake8 {posargs} bandit"},{"line_number":34,"context_line":"           flake8 {posargs} tests"},{"line_number":35,"context_line":"           bandit-baseline -r bandit -ll -ii"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"[testenv:venv]"},{"line_number":38,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7a77a97e_6316eca8","line":35,"in_reply_to":"7a77a97e_3d2a57e8","updated":"2016-11-21 20:08:02.000000000","message":"Done","commit_id":"bd621bf49cdc6e672704bac0ddfe17ae91c18ac5"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"e5d9887a583ea108580105851e6aa5c324e7b1a6","unresolved":false,"context_lines":[{"line_number":32,"context_line":"       ."},{"line_number":33,"context_line":"commands \u003d flake8 {posargs} bandit"},{"line_number":34,"context_line":"           flake8 {posargs} tests"},{"line_number":35,"context_line":"           bandit-baseline -r bandit -ll -ii"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"[testenv:venv]"},{"line_number":38,"context_line":"commands \u003d {posargs}"}],"source_content_type":"text/x-properties","patch_set":2,"id":"7a77a97e_7d0e8fe6","line":35,"in_reply_to":"7a77a97e_dd0fa32c","updated":"2016-11-21 18:56:47.000000000","message":"Well, I think that\u0027s a good idea, but currently the pylint job is not clean.  Its showing many errors.  So do you guys prefer to incrementally fixing these or a mega-patch to fix and/ignore pylint errors?","commit_id":"bd621bf49cdc6e672704bac0ddfe17ae91c18ac5"}]}