)]}'
{"tox.ini":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"74616095cbf2c898b487b9b94fe71a085fce5ea5","unresolved":false,"context_lines":[{"line_number":20,"context_line":"commands \u003d {toxinidir}/wrap_nosetests.sh --no-path-adjustment --with-coverage --cover-erase --cover-package\u003dceilometer --cover-inclusive []"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[testenv:pep8]"},{"line_number":23,"context_line":"deps \u003d -r{toxinidir}/tools/test-requires"},{"line_number":24,"context_line":"       -r{toxinidir}/tools/pip-requires"},{"line_number":25,"context_line":"       pep8\u003d\u003d1.3.3"},{"line_number":26,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAM3%2F%2B0sM%3D","line":23,"updated":"2013-02-18 14:57:54.000000000","message":"Is this really related to this changes? I thought the default venv would take care of that.","commit_id":"5181724ecea74117c1d3effe4ccb827b113f1662"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"45d0b590542a09e3e624743e043353c2de80ea96","unresolved":false,"context_lines":[{"line_number":20,"context_line":"commands \u003d {toxinidir}/wrap_nosetests.sh --no-path-adjustment --with-coverage --cover-erase --cover-package\u003dceilometer --cover-inclusive []"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[testenv:pep8]"},{"line_number":23,"context_line":"deps \u003d -r{toxinidir}/tools/test-requires"},{"line_number":24,"context_line":"       -r{toxinidir}/tools/pip-requires"},{"line_number":25,"context_line":"       pep8\u003d\u003d1.3.3"},{"line_number":26,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAM3%2F%2B0m4%3D","line":23,"in_reply_to":"AAAAM3%2F%2B0nQ%3D","updated":"2013-02-18 15:43:31.000000000","message":"Ah thanks for the explanation. :)","commit_id":"5181724ecea74117c1d3effe4ccb827b113f1662"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"d7eccef7c9107bfac30800bd66073808f15b30dc","unresolved":false,"context_lines":[{"line_number":20,"context_line":"commands \u003d {toxinidir}/wrap_nosetests.sh --no-path-adjustment --with-coverage --cover-erase --cover-package\u003dceilometer --cover-inclusive []"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[testenv:pep8]"},{"line_number":23,"context_line":"deps \u003d -r{toxinidir}/tools/test-requires"},{"line_number":24,"context_line":"       -r{toxinidir}/tools/pip-requires"},{"line_number":25,"context_line":"       pep8\u003d\u003d1.3.3"},{"line_number":26,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"AAAAM3%2F%2B0nQ%3D","line":23,"in_reply_to":"AAAAM3%2F%2B0sM%3D","updated":"2013-02-18 15:36:52.000000000","message":"Click on the changeid for this review and look at the corresponding glance one\n\nClark Boylan debugged the issue and suggested removing the deps from tox.ini and moving them to test-requires - in the case of other projects, that\u0027s fine because they already have pep8 in test-requires, but ceilometer doesn\u0027t ... so I went with this option","commit_id":"5181724ecea74117c1d3effe4ccb827b113f1662"}]}