)]}'
{"tools/check-json-syntax.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"56a572bc766bbd5cb6e173f2dce999a7220d6aa4","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"def main():"},{"line_number":111,"context_line":"    # souce directory for JSON files."},{"line_number":112,"context_line":"    path \u003d sys.argv[1]"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"    exit_status \u003d 0"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_3fe66cb4","line":111,"range":{"start_line":111,"start_character":6,"end_line":111,"end_character":11},"updated":"2016-05-25 13:53:01.000000000","message":"/souce/source/","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"1332c5f4c0ba81f9ea684e90c5f6f2ef9f88fd57","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"def main():"},{"line_number":111,"context_line":"    # souce directory for JSON files."},{"line_number":112,"context_line":"    path \u003d sys.argv[1]"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"    exit_status \u003d 0"}],"source_content_type":"text/x-python","patch_set":3,"id":"9abb7d3a_071101ad","line":111,"range":{"start_line":111,"start_character":6,"end_line":111,"end_character":11},"in_reply_to":"bab6814e_3fe66cb4","updated":"2016-05-27 11:07:38.000000000","message":"Done","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"}],"tox.ini":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"56a572bc766bbd5cb6e173f2dce999a7220d6aa4","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[testenv:api-ref]"},{"line_number":29,"context_line":"# (sheel)This environment is called from CI scripts to test and publish"},{"line_number":30,"context_line":"# the API Ref to developer.openstack.org."},{"line_number":31,"context_line":"whitelist_externals \u003d rm"},{"line_number":32,"context_line":"deps \u003d -r{toxinidir}/test-requirements.txt"},{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"bab6814e_dfc5a03d","line":30,"updated":"2016-05-25 13:53:01.000000000","message":"Might be good to add \"envdir \u003d {toxworkdir}/pep8\" so it reuses the same venv as the pep8 job to limit the number of virtual environments created. Or maybe better \"envdir \u003d {toxworkdir}/docs\"","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"1332c5f4c0ba81f9ea684e90c5f6f2ef9f88fd57","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"[testenv:api-ref]"},{"line_number":29,"context_line":"# (sheel)This environment is called from CI scripts to test and publish"},{"line_number":30,"context_line":"# the API Ref to developer.openstack.org."},{"line_number":31,"context_line":"whitelist_externals \u003d rm"},{"line_number":32,"context_line":"deps \u003d -r{toxinidir}/test-requirements.txt"},{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9abb7d3a_270c0594","line":30,"in_reply_to":"bab6814e_dfc5a03d","updated":"2016-05-27 11:07:38.000000000","message":"Done","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"56a572bc766bbd5cb6e173f2dce999a7220d6aa4","unresolved":false,"context_lines":[{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"},{"line_number":34,"context_line":"commands \u003d"},{"line_number":35,"context_line":"  rm -rf cinder/api-ref/build"},{"line_number":36,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v1/source/samples/"},{"line_number":37,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v2/source/samples/"},{"line_number":38,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v1 api-ref/v1/source api-ref/build/html/v1"},{"line_number":39,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v2 api-ref/v2/source api-ref/build/html/v2"}],"source_content_type":"text/x-properties","patch_set":3,"id":"bab6814e_7fc75412","line":36,"updated":"2016-05-25 13:53:01.000000000","message":"We could find all json files similar to how the current pep8 job is finding all .po and .pot files.","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"e5d305e50129a8f595d918a089ee280c96590bc6","unresolved":false,"context_lines":[{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"},{"line_number":34,"context_line":"commands \u003d"},{"line_number":35,"context_line":"  rm -rf cinder/api-ref/build"},{"line_number":36,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v1/source/samples/"},{"line_number":37,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v2/source/samples/"},{"line_number":38,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v1 api-ref/v1/source api-ref/build/html/v1"},{"line_number":39,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v2 api-ref/v2/source api-ref/build/html/v2"}],"source_content_type":"text/x-properties","patch_set":3,"id":"7aa08908_86f5e984","line":36,"in_reply_to":"9abb7d3a_605ffc75","updated":"2016-06-06 03:45:53.000000000","message":"Dear Sean,\nyes you are right, its ok to use this in current way as it seems clean to use this seperately..","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"4c53c5776c856d09a93c522cac1165a4b30d5bf3","unresolved":false,"context_lines":[{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"},{"line_number":34,"context_line":"commands \u003d"},{"line_number":35,"context_line":"  rm -rf cinder/api-ref/build"},{"line_number":36,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v1/source/samples/"},{"line_number":37,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v2/source/samples/"},{"line_number":38,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v1 api-ref/v1/source api-ref/build/html/v1"},{"line_number":39,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v2 api-ref/v2/source api-ref/build/html/v2"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9abb7d3a_605ffc75","line":36,"in_reply_to":"9abb7d3a_c716d9a6","updated":"2016-06-02 18:15:48.000000000","message":"My point was to reduce this to one line instead of two with the only difference being v1 or v2, but that\u0027s fine as it is.\n\nSounds like it would be easy enough to just pass the path of \"api-ref/\", but perhaps that would pick up more than we would really want.","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"1332c5f4c0ba81f9ea684e90c5f6f2ef9f88fd57","unresolved":false,"context_lines":[{"line_number":33,"context_line":"install_command \u003d pip install -U --force-reinstall {opts} {packages}"},{"line_number":34,"context_line":"commands \u003d"},{"line_number":35,"context_line":"  rm -rf cinder/api-ref/build"},{"line_number":36,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v1/source/samples/"},{"line_number":37,"context_line":"  {toxinidir}/tools/check-json-syntax.py api-ref/v2/source/samples/"},{"line_number":38,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v1 api-ref/v1/source api-ref/build/html/v1"},{"line_number":39,"context_line":"  sphinx-build -W -b html -d api-ref/build/doctrees/v2 api-ref/v2/source api-ref/build/html/v2"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9abb7d3a_c716d9a6","line":36,"in_reply_to":"bab6814e_7fc75412","updated":"2016-05-27 11:07:38.000000000","message":"This check is implemented in check-json-syntax.py at Line 118\n\njson_files \u003d [f for f in files if f.endswith(\u0027.json\u0027)]","commit_id":"65b485d23aceb74213e0b9484f1b804bbadd8047"}]}