)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19930,"name":"Dai Dang Van","email":"daidv@omzcloud.vn","username":"daidv"},"change_message_id":"1890248e40202603989888c68eaba710079eff41","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nguyen Van Trung \u003ctrungnv@vn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-11-16 14:57:34 +0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add validation to improve pep8 check for designate project"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This PS will help project avoid pep8 some issues for *.rst"},{"line_number":10,"context_line":"in whole project."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f485f77_bd8798b5","line":7,"range":{"start_line":7,"start_character":41,"end_line":7,"end_character":50},"updated":"2017-11-16 09:15:40.000000000","message":"You may want to see glance instead of designate in this.","commit_id":"3af154d7b9d39468286c4f4c2968f52dde5f3e26"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4762cb4cbb5c2bcb143ffb2986c9800d95f81900","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add validation to improve pep8 check for glance project"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This PS will help project avoid pep8 some issues for *.rst"},{"line_number":10,"context_line":"in whole project."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Co-authored-By: Hoang Trung Hieu \u003chieuht@vn.fujitsu.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ff82abbf_43a210fe","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":36},"updated":"2017-11-27 13:44:36.000000000","message":"If you do an update, might be good to state that you are adding doc8 rather than saying pep8.","commit_id":"a24398f11f71e9da165d2fa89f093bde5bc861a2"}],"tox.ini":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6a6c84519a1df0b6f5a4be77c80d1d7673116280","unresolved":false,"context_lines":[{"line_number":59,"context_line":"  bandit -c bandit.yaml -r glance -n5 -p gate"},{"line_number":60,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":61,"context_line":"  bash -c \"find glance -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":62,"context_line":"  doc8 README.rst releasenotes/source doc/source rally-jobs CONTRIBUTING.rst --ignore D000"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"[testenv:genconfig]"},{"line_number":65,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":7,"id":"ff82abbf_70b319ea","line":62,"updated":"2017-11-29 14:53:12.000000000","message":"Instead of listing the docs to check, let\u0027s do excludes like for flake8.\n\nI think this line can just be \u0027doc8\u0027 with nothing else (see below).","commit_id":"d08744040a3977afdbd5cb00e24df37f35071af1"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"01743c10a6059079eb8a6fb7bff86c9511462de3","unresolved":false,"context_lines":[{"line_number":59,"context_line":"  bandit -c bandit.yaml -r glance -n5 -p gate"},{"line_number":60,"context_line":"  # Check that .po and .pot files are valid:"},{"line_number":61,"context_line":"  bash -c \"find glance -type f -regex \u0027.*\\.pot?\u0027 -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null\""},{"line_number":62,"context_line":"  doc8 README.rst releasenotes/source doc/source rally-jobs CONTRIBUTING.rst --ignore D000"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"[testenv:genconfig]"},{"line_number":65,"context_line":"commands \u003d"}],"source_content_type":"text/x-properties","patch_set":7,"id":"ff82abbf_01427dab","line":62,"in_reply_to":"ff82abbf_70b319ea","updated":"2017-11-29 16:28:55.000000000","message":"Done","commit_id":"d08744040a3977afdbd5cb00e24df37f35071af1"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6a6c84519a1df0b6f5a4be77c80d1d7673116280","unresolved":false,"context_lines":[{"line_number":89,"context_line":"deps \u003d bindep"},{"line_number":90,"context_line":"commands \u003d bindep test"},{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[flake8]"},{"line_number":94,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"},{"line_number":95,"context_line":"# E711  comparison to None should be \u0027if cond is not None:\u0027"}],"source_content_type":"text/x-properties","patch_set":7,"id":"ff82abbf_50e5b5da","line":92,"updated":"2017-11-29 14:53:12.000000000","message":"add a new section here:\n\n  [doc8]\n  ignore \u003d D000\n  ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*\n\nYou could add a comment explaining why we\u0027re ignoring D000.  (Why are we ignoring it?)","commit_id":"d08744040a3977afdbd5cb00e24df37f35071af1"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"01743c10a6059079eb8a6fb7bff86c9511462de3","unresolved":false,"context_lines":[{"line_number":89,"context_line":"deps \u003d bindep"},{"line_number":90,"context_line":"commands \u003d bindep test"},{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[flake8]"},{"line_number":94,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"},{"line_number":95,"context_line":"# E711  comparison to None should be \u0027if cond is not None:\u0027"}],"source_content_type":"text/x-properties","patch_set":7,"id":"ff82abbf_a11631a7","line":92,"in_reply_to":"ff82abbf_50e5b5da","updated":"2017-11-29 16:28:55.000000000","message":"Done","commit_id":"d08744040a3977afdbd5cb00e24df37f35071af1"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"3366f51144757f40655887d2edb08e1f1c0226f4","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":9,"id":"ff82abbf_31a9fa8f","line":94,"updated":"2017-11-30 13:34:05.000000000","message":"Looks like doc/source/api and doc/build need to be excluded as well. I get 293 errors running this as is.","commit_id":"7dc938409c5dccba66282a89dab8568d68bea804"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"d9cdfb3b2738e3d53d107b25a97a9b87286ede3d","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":9,"id":"ff82abbf_9c8d7d33","line":94,"in_reply_to":"ff82abbf_31a9fa8f","updated":"2017-11-30 14:10:04.000000000","message":"No. /api and /build both are result when you generate docs via tox -edocs. We don\u0027t need these dir in ignore-path of doc8.","commit_id":"7dc938409c5dccba66282a89dab8568d68bea804"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"948d3a2ac2ceea2f8a764fd384bebae2d41ab85a","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":9,"id":"ff82abbf_fca04994","line":94,"in_reply_to":"ff82abbf_9c8d7d33","updated":"2017-11-30 14:10:55.000000000","message":"Except that you do, because if you run tox -e docs before you run this you end up with 293 failures.","commit_id":"7dc938409c5dccba66282a89dab8568d68bea804"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"96cae27a00fc9aa7aa39b2afb11ecc09eb180300","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":9,"id":"ff82abbf_4ca9808e","line":94,"in_reply_to":"ff82abbf_fca04994","updated":"2017-11-30 15:42:59.000000000","message":"Yep. I will update this change in new PS. Thanks.","commit_id":"7dc938409c5dccba66282a89dab8568d68bea804"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"acd9b0cabef0c743bf85d2ea967a1e6b60e18fe4","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*,doc/source/api,doc/build"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":11,"id":"df87a7cf_4163c5b2","line":94,"range":{"start_line":94,"start_character":71,"end_line":94,"end_character":85},"updated":"2017-12-11 03:37:21.000000000","message":"This directory doesn\u0027t exist.  Might be a good idea to exclude api-ref/build, though.  (It doesn\u0027t seem to have any errors, but might as well exclude all build directories.  We definitely want to process all the source dirs, though.)","commit_id":"4ade280e0cc45f8090d00412224bfda518b61be4"},{"author":{"_account_id":25254,"name":"Nguyen Van Trung","email":"trungnvfet@outlook.com","username":"trungnv"},"change_message_id":"ce653673c7fec8504e57782a5f75cde65c6ff714","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*,doc/source/api,doc/build"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":11,"id":"df87a7cf_c124b560","line":94,"range":{"start_line":94,"start_character":71,"end_line":94,"end_character":85},"in_reply_to":"df87a7cf_4163c5b2","updated":"2017-12-11 03:45:41.000000000","message":"Done","commit_id":"4ade280e0cc45f8090d00412224bfda518b61be4"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c873d58e895a832fcd086c4fb4f611de815b53f8","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*,doc/source/api,doc/build"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":11,"id":"df87a7cf_95c31d38","line":94,"range":{"start_line":94,"start_character":71,"end_line":94,"end_character":85},"in_reply_to":"df87a7cf_c124b560","updated":"2017-12-12 15:52:50.000000000","message":"doc/source/api is dynamically generated during doc builds, so that does still need to be in the ignore list.","commit_id":"4ade280e0cc45f8090d00412224bfda518b61be4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d32dfdcdeab02e8e2d85a12f5e5c9275218b4204","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*,api-ref/build,doc/build,doc/source/api"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":16,"id":"7f96bb07_0d5dcb45","line":94,"range":{"start_line":94,"start_character":95,"end_line":94,"end_character":109},"updated":"2018-01-18 23:44:24.000000000","message":"I don\u0027t see this generated, but what the heck.  I do see doc/source/contributor/api being generated, though, so let\u0027s add that one and we\u0027ll be done with this.\n\n.gitignore mentions both doc/source/api and doc/source/contributor/api as being created by the docs build, so that\u0027s a good for me!","commit_id":"60cabdfa5b7885125d00dc26da9120553dd883a6"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"bd616ce8912fab375575cd5ad1102901046f5e10","unresolved":false,"context_lines":[{"line_number":91,"context_line":"usedevelop \u003d False"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"[doc8]"},{"line_number":94,"context_line":"ignore-path \u003d .venv,.git,.tox,*glance/locale*,*lib/python*,glance.egg*,api-ref/build,doc/build,doc/source/api"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"[flake8]"},{"line_number":97,"context_line":"# TODO(dmllr): Analyze or fix the warnings blacklisted below"}],"source_content_type":"text/x-properties","patch_set":16,"id":"7f96bb07_415dd4e7","line":94,"range":{"start_line":94,"start_character":95,"end_line":94,"end_character":109},"in_reply_to":"7f96bb07_0d5dcb45","updated":"2018-01-19 08:18:59.000000000","message":"setup.cfg has:\napi_doc_dir \u003d contributor/api\n\nSo, it\u0027s really only doc/source/contributor/api","commit_id":"60cabdfa5b7885125d00dc26da9120553dd883a6"}]}