)]}'
{"doc/requirements.txt":[{"author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"change_message_id":"1b06d125e969b1bbdb1ed98af1e2a1f4419c8f95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"df87a7cf_579bb916","line":13,"updated":"2017-12-19 07:59:47.000000000","message":"please add ldap\u0027s related lib as well.","commit_id":"fbeff12dd11331a1fd614ba50dd00c19602a871e"}],"tox.ini":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"51eb9a8da1b96e8f95f0cd331d682c8f7c8dc1d9","unresolved":false,"context_lines":[{"line_number":128,"context_line":"[testenv:releasenotes]"},{"line_number":129,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"},{"line_number":130,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"[testenv:genconfig]"},{"line_number":133,"context_line":"commands \u003d oslo-config-generator --config-file\u003dconfig-generator/keystone.conf"},{"line_number":134,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"df87a7cf_01af26e3","line":131,"updated":"2017-12-19 12:14:10.000000000","message":"I think these two changes aren\u0027t needed, at least this patch still seems to work for me without them.","commit_id":"ef9308cc04f8a7698a033ac82357b4664a0adfd2"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"c992e51f9da001355a03c5ca41d3b33ed3dbb112","unresolved":false,"context_lines":[{"line_number":119,"context_line":"max-complexity\u003d24"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"[testenv:docs]"},{"line_number":122,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":123,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"[testenv:releasenotes]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"df87a7cf_a1bd1ab8","line":122,"updated":"2017-12-19 12:29:39.000000000","message":"I think this shouldn\u0027t be changed yet, you can see on https://review.openstack.org/#/c/528796/4/roles/sphinx/tasks/main.yaml@35 that they actually reverted it to what we were using before because the autodoc feature doesn\u0027t work with this","commit_id":"3669063f0c6eaa5a3be2b5c59e291f53d61833dd"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"3e83c0975d526394a2a3ed8ee3d3b3ca01f1e432","unresolved":false,"context_lines":[{"line_number":119,"context_line":"max-complexity\u003d24"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"[testenv:docs]"},{"line_number":122,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":123,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"[testenv:releasenotes]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"df87a7cf_d6e6e116","line":122,"in_reply_to":"df87a7cf_a1bd1ab8","updated":"2017-12-19 15:17:16.000000000","message":"Agreed, this should be changed back","commit_id":"3669063f0c6eaa5a3be2b5c59e291f53d61833dd"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"c992e51f9da001355a03c5ca41d3b33ed3dbb112","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"[testenv:releasenotes]"},{"line_number":126,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"},{"line_number":127,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"[testenv:genconfig]"},{"line_number":130,"context_line":"commands \u003d oslo-config-generator --config-file\u003dconfig-generator/keystone.conf"}],"source_content_type":"text/x-properties","patch_set":4,"id":"df87a7cf_01cc0668","line":127,"updated":"2017-12-19 12:29:39.000000000","message":"Not sure this is needed, tox -ereleasenotes works for me without it.","commit_id":"3669063f0c6eaa5a3be2b5c59e291f53d61833dd"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"3e83c0975d526394a2a3ed8ee3d3b3ca01f1e432","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"[testenv:releasenotes]"},{"line_number":126,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"},{"line_number":127,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"[testenv:genconfig]"},{"line_number":130,"context_line":"commands \u003d oslo-config-generator --config-file\u003dconfig-generator/keystone.conf"}],"source_content_type":"text/x-properties","patch_set":4,"id":"df87a7cf_76c11554","line":127,"in_reply_to":"df87a7cf_01cc0668","updated":"2017-12-19 15:17:16.000000000","message":"Colleen, where do you get sphinx from in that case?\n\nThe above is what we do in the job, so let\u0027s go with it...","commit_id":"3669063f0c6eaa5a3be2b5c59e291f53d61833dd"}]}