)]}'
{"HACKING.rst":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5c59091df511d129bb417a0f1652d9b5469635bd","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Use tox -eautopep8 before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5fc1f717_58d61ec6","line":31,"updated":"2019-03-20 01:11:56.000000000","message":"Do we encourage contributors to use autopep8?\nAccording to http://codesearch.openstack.org/?q\u003dautopep8\u0026i\u003dnope\u0026files\u003d\u0026repos\u003d autopep8 doesn\u0027t seem majority in OpenStack and we are using `tox -epep8 -vv` at the gate.\nSo I am not sure this way is valid for us at this time.","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"f0c04660d5ba559c0e0b7b58f7a921af40f86f7d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Use tox -eautopep8 before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5fc1f717_1cec45b9","line":31,"updated":"2019-03-28 05:25:13.000000000","message":"yeah, we don\u0027t need to install autopep8 explicitly as afazekas mentioned. But I feel it\u0027s good to rephrase like Felipe suggestion without \"If you have autopep8 install,\" since we don\u0027t need to force to use autopep8 for everybody.","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"b6e53ca862449460217634fbe8d4e00df057fc30","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Use tox -eautopep8 before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5fc1f717_bc4edf35","line":31,"in_reply_to":"5fc1f717_58d61ec6","updated":"2019-03-22 00:33:39.000000000","message":"Maybe we can rephrase:\n\n\"If you have autopep8 install, you can run `tox -e autopep8` before submitting a patch to make sure it passes all lint/hacking checks easily.\"","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"32821b040016ef31fc4671bc9eb9fde5dad2c0c1","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Use tox -eautopep8 before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5fc1f717_7c75d297","line":31,"in_reply_to":"5fc1f717_7c62b7af","updated":"2019-03-22 07:11:52.000000000","message":"It is tox, you will have autopep8, because it is defined in the environment.","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"dc6ffe80445b149800527de570f66db822e8753d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Use tox -eautopep8 before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5fc1f717_7c62b7af","line":31,"in_reply_to":"5fc1f717_bc4edf35","updated":"2019-03-22 00:34:24.000000000","message":"s/If you have autopep8 install/If you have autopep8 installed","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"f49b31700bbfe7412238a3ef140e27b26f1263f2","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- [T117] Check negative tests have ``@decorators.attr(type\u003d[\u0027negative\u0027])``"},{"line_number":29,"context_line":"  applied."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Recomended to use ``tox -eautopep8`` before submitting a patch."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Test Data/Configuration"},{"line_number":34,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5fc1f717_62257cae","line":31,"range":{"start_line":31,"start_character":0,"end_line":31,"end_character":10},"updated":"2019-03-29 17:28:58.000000000","message":"Recommended","commit_id":"cb17798b3785d91a74b7bbe3205e80a2b1a7f24b"}],"tools/format.sh":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"b6e53ca862449460217634fbe8d4e00df057fc30","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    AUTOPEP8\u003d`which autopep8-3`"},{"line_number":9,"context_line":"fi"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"if [[ -z \"$AUTOPEP8\" ]]; then"},{"line_number":12,"context_line":"    echo \"Unable to locate autopep8\" \u003e\u00262"},{"line_number":13,"context_line":"    exit 2"},{"line_number":14,"context_line":"fi"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"# isort is not compatible with the default flake8 (H306), maybe flake8-isort"},{"line_number":17,"context_line":"# isort -rc -sl -fss ../tempest ../setup.py"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5fc1f717_dc496b32","line":14,"range":{"start_line":11,"start_character":0,"end_line":14,"end_character":2},"updated":"2019-03-22 00:33:39.000000000","message":"I agree with Keni\u0027chi because if it\u0027s not very common then the user needs to install just to run autopep8. See earlier comment...","commit_id":"cb03b10f2588305f95f333d71a36c935ab891e9c"}],"tox.ini":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"f0d4f21cd6816b5adb7bb09a935d4597a8f49c8a","unresolved":false,"context_lines":[{"line_number":199,"context_line":"[testenv:pep8]"},{"line_number":200,"context_line":"deps \u003d"},{"line_number":201,"context_line":"    -r{toxinidir}/test-requirements.txt"},{"line_number":202,"context_line":"    autopep8"},{"line_number":203,"context_line":"basepython \u003d python3"},{"line_number":204,"context_line":"commands \u003d"},{"line_number":205,"context_line":"    autopep8 --exit-code --max-line-length\u003d79 --experimental --diff -r tempest setup.py"}],"source_content_type":"text/x-properties","patch_set":4,"id":"5fc1f717_a4436113","line":202,"updated":"2019-04-01 09:43:16.000000000","message":"+1","commit_id":"c07edb5f8072e238caa0dfbf38bb719c6ab20c12"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aa6ae1ca94bb497b2ed098df4fda96a5aba52410","unresolved":false,"context_lines":[{"line_number":199,"context_line":"[testenv:pep8]"},{"line_number":200,"context_line":"deps \u003d"},{"line_number":201,"context_line":"    -r{toxinidir}/test-requirements.txt"},{"line_number":202,"context_line":"    autopep8"},{"line_number":203,"context_line":"basepython \u003d python3"},{"line_number":204,"context_line":"commands \u003d"},{"line_number":205,"context_line":"    autopep8 --exit-code --max-line-length\u003d79 --experimental --diff -r tempest setup.py"}],"source_content_type":"text/x-properties","patch_set":4,"id":"5fc1f717_8c2a1a7c","line":202,"range":{"start_line":202,"start_character":4,"end_line":202,"end_character":12},"updated":"2019-03-29 20:51:46.000000000","message":"I am thinking we should have some version cap for this in case future version does not break us.\n\nBut that should be done in sepatrate patch is needed, not in this which is more documentation one.","commit_id":"c07edb5f8072e238caa0dfbf38bb719c6ab20c12"}]}