)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"41cb797ff34f811f683a84871d06f640776fa9ef","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Client object to documentation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After wanting to do a rather simple thing -- figure out of the"},{"line_number":10,"context_line":"\"timeout\" argument to nodepool was a int or a float, it lead me down"},{"line_number":11,"context_line":"the rabbit-hole to python-novaclient."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a80dd14_c1a9d37c","line":9,"updated":"2015-03-11 14:24:25.000000000","message":"First two paragraphs looks like not related to the problem","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e4eea7e771cabb7f37a1800df2dde0803433250d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Client object to documentation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After wanting to do a rather simple thing -- figure out of the"},{"line_number":10,"context_line":"\"timeout\" argument to nodepool was a int or a float, it lead me down"},{"line_number":11,"context_line":"the rabbit-hole to python-novaclient."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a80dd14_7b5a752b","line":9,"in_reply_to":"9a80dd14_c1a9d37c","updated":"2015-03-12 00:43:19.000000000","message":"i\u0027m just trying to motivate the change","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"41cb797ff34f811f683a84871d06f640776fa9ef","unresolved":false,"context_lines":[{"line_number":17,"context_line":"it\u0027s not included in the class list.  So add that and also at least"},{"line_number":18,"context_line":"document the types of the arguments."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"However, then I noticd that this wasn\u0027t showing up; turns out sphinx"},{"line_number":21,"context_line":"requires \"autoclass_content \u003d both\" if you want it to document"},{"line_number":22,"context_line":"__init__() functions.  Several other classes had their init args"},{"line_number":23,"context_line":"documented but they weren\u0027t showing up because of this."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a80dd14_01304b55","line":20,"updated":"2015-03-11 14:24:25.000000000","message":"Not totally sure if we need such content at all","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e4eea7e771cabb7f37a1800df2dde0803433250d","unresolved":false,"context_lines":[{"line_number":17,"context_line":"it\u0027s not included in the class list.  So add that and also at least"},{"line_number":18,"context_line":"document the types of the arguments."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"However, then I noticd that this wasn\u0027t showing up; turns out sphinx"},{"line_number":21,"context_line":"requires \"autoclass_content \u003d both\" if you want it to document"},{"line_number":22,"context_line":"__init__() functions.  Several other classes had their init args"},{"line_number":23,"context_line":"documented but they weren\u0027t showing up because of this."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a80dd14_bb3b0dc7","line":20,"in_reply_to":"9a80dd14_01304b55","updated":"2015-03-12 00:43:19.000000000","message":"well, having to dig through the code to figure out what types can be passed to the class initializer isn\u0027t a good UI IMO","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"}],"doc/source/conf.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"58bffc16475988cc79271e531f5a7cc6d7d13fdb","unresolved":false,"context_lines":[{"line_number":86,"context_line":"# coming with Sphinx (named \u0027sphinx.ext.*\u0027) or your custom ones."},{"line_number":87,"context_line":"extensions \u003d [\u0027sphinx.ext.autodoc\u0027, \u0027oslosphinx\u0027]"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"autoclass_content \u003d \u0027both\u0027"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# Add any paths that contain templates here, relative to this directory."},{"line_number":92,"context_line":"templates_path \u003d [\u0027_templates\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_4d943244","line":89,"updated":"2015-03-11 15:26:28.000000000","message":"So, I can\u0027t find this option being set in python-glanceclient or python-keystoneclient.  Are we certain this is necessary?","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e4eea7e771cabb7f37a1800df2dde0803433250d","unresolved":false,"context_lines":[{"line_number":86,"context_line":"# coming with Sphinx (named \u0027sphinx.ext.*\u0027) or your custom ones."},{"line_number":87,"context_line":"extensions \u003d [\u0027sphinx.ext.autodoc\u0027, \u0027oslosphinx\u0027]"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"autoclass_content \u003d \u0027both\u0027"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# Add any paths that contain templates here, relative to this directory."},{"line_number":92,"context_line":"templates_path \u003d [\u0027_templates\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_9bcd6989","line":89,"in_reply_to":"9a80dd14_4d943244","updated":"2015-03-12 00:43:19.000000000","message":"it is necessary to document the __init__ of Client.  it also follows classes; take a look at [1] for example v [2].  it shows you the documentation for the base-class\n\n[1] http://docs.openstack.org/developer/python-novaclient/ref/v2/quotas.html\n\n[2] http://docs-draft.openstack.org/80/163280/1/check/gate-python-novaclient-docs/3273929//doc/build/html/ref/v2/quotas.html","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"}],"novaclient/v2/client.py":[{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"41cb797ff34f811f683a84871d06f640776fa9ef","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                 connection_pool\u003dFalse, session\u003dNone, auth\u003dNone,"},{"line_number":106,"context_line":"                 **kwargs):"},{"line_number":107,"context_line":"        \"\"\""},{"line_number":108,"context_line":"        :param str username:"},{"line_number":109,"context_line":"        :param str api_key:"},{"line_number":110,"context_line":"        :param str project_id:"},{"line_number":111,"context_line":"        :param str auth_url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_c1c47361","line":108,"updated":"2015-03-11 14:24:25.000000000","message":"I think such doc isn\u0027t informative enough to be in. Can you fill this fields with any information about args?","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e4eea7e771cabb7f37a1800df2dde0803433250d","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                 connection_pool\u003dFalse, session\u003dNone, auth\u003dNone,"},{"line_number":106,"context_line":"                 **kwargs):"},{"line_number":107,"context_line":"        \"\"\""},{"line_number":108,"context_line":"        :param str username:"},{"line_number":109,"context_line":"        :param str api_key:"},{"line_number":110,"context_line":"        :param str project_id:"},{"line_number":111,"context_line":"        :param str auth_url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_db752133","line":108,"in_reply_to":"9a80dd14_c1c47361","updated":"2015-03-12 00:43:19.000000000","message":"I think the types are at least a start.  I can fill it in with terse descriptions but people who know the code can do a better job to follow on","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"58bffc16475988cc79271e531f5a7cc6d7d13fdb","unresolved":false,"context_lines":[{"line_number":133,"context_line":"        :param bool connection_pool:"},{"line_number":134,"context_line":"        :param str session:"},{"line_number":135,"context_line":"        :param str auth:"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":"        # FIXME(comstud): Rename the api_key argument above when we"},{"line_number":139,"context_line":"        # know it\u0027s not being used as keyword argument"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_0d26ba68","line":136,"updated":"2015-03-11 15:26:28.000000000","message":"The trailing blank line in a docstring is no longer necessary and can be eliminated.","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e4eea7e771cabb7f37a1800df2dde0803433250d","unresolved":false,"context_lines":[{"line_number":133,"context_line":"        :param bool connection_pool:"},{"line_number":134,"context_line":"        :param str session:"},{"line_number":135,"context_line":"        :param str auth:"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":"        # FIXME(comstud): Rename the api_key argument above when we"},{"line_number":139,"context_line":"        # know it\u0027s not being used as keyword argument"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_9b948945","line":136,"in_reply_to":"9a80dd14_0d26ba68","updated":"2015-03-12 00:43:19.000000000","message":"Done","commit_id":"32da111c5c351a6092f479f84014647d432af5a2"}]}