)]}'
{"designate/backend/impl_akamai.py":[{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"5f6871b84b808b0f138032095c34921ffed0d4f9","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    `Docs \u003cbackend.html\u003e`_"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"\"\"\"  # noqa"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"import os"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_12d00f53","line":24,"range":{"start_line":24,"start_character":5,"end_line":24,"end_character":11},"updated":"2016-05-06 12:58:03.000000000","message":"Is this needed? If it is, we\u0027re Doing It Wrong™.","commit_id":"11942791bf2fcc0e15d4635fcaca2d9682aed583"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"06acdfd9ed3e9e6b92deb3eac04abb06e76892c0","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    `Docs \u003cbackend.html\u003e`_"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"\"\"\"  # noqa"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"import os"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dab17558_e84da0c5","line":24,"updated":"2016-05-12 13:49:15.000000000","message":"this is a separate change","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"58c8c7db17e5abfa9f0cf88c3e1cd9df3f3de990","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    `Docs \u003cbackend.html\u003e`_"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"\"\"\"  # noqa"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"import os"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dab17558_447e95e4","line":24,"in_reply_to":"dab17558_e84da0c5","updated":"2016-05-17 12:56:07.000000000","message":"Done","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"}],"doc/source/developer-guidelines.rst":[{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"2b3387e7d4a86270fb3d338298154df2378f2351","unresolved":false,"context_lines":[{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a18252c_9189b9cd","line":53,"updated":"2016-04-12 13:11:18.000000000","message":"I personally think we should be sticking to the openstack/hacking guide, as we actually test for that in the gate.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"c25121fc2caed7fc649f77b5f51dab5ad0b94200","unresolved":false,"context_lines":[{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_f32bd492","line":53,"updated":"2016-04-08 16:37:02.000000000","message":"We should just point to the openstack style guide, not that there\u0027s really anything wrong with the pocoo one.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"cfcb970f301c3811dd46b31837410c4a07025bc4","unresolved":false,"context_lines":[{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a18252c_2ed92d84","line":53,"in_reply_to":"5a18252c_9189b9cd","updated":"2016-04-13 09:56:55.000000000","message":"Agree, lets stick with what the OpenStack community does here. If there are particular gaps in the OS guides that Pocoo covers, we can discuss them and add hacking checks.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"3357313cb6cacc510b92fe632c7eb8dccc942159","unresolved":false,"context_lines":[{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a1b2928_4e65b743","line":53,"in_reply_to":"7a1b2928_3926d568","updated":"2016-04-11 14:49:11.000000000","message":"It seems weird pointing to two style guides, I guess. Pocoo is outside of openstack. What if the pocoo guide changes or the two guides conflict on something? \n\nI mean, it\u0027s not a big deal. I\u0027m fine with leaving it in, if we want.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"4c442451ce04100e2f1ab38766abeecf9f97be73","unresolved":false,"context_lines":[{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a1b2928_3926d568","line":53,"in_reply_to":"9a061dce_f32bd492","updated":"2016-04-11 11:26:58.000000000","message":"\u003e We should just point to the openstack style guide\n\nCare to explain why? The Pocoo Style guide is listed as optional and contains hints that people largely follow already (good indentation, spacing, comments, naming...)","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"c25121fc2caed7fc649f77b5f51dab5ad0b94200","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":".. code-block:: python"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    # -*- coding: utf-8 -*-"},{"line_number":63,"context_line":"    # Copyright \u003cyear\u003e \u003ccompany\u003e"},{"line_number":64,"context_line":"    #"},{"line_number":65,"context_line":"    # Author: \u003cname\u003e \u003cemail addr\u003e"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_9681a6fe","line":62,"updated":"2016-04-08 16:37:02.000000000","message":"Do we want these encoding declarations? I think they\u0027re unnecessary and Designate hasn\u0027t been using them so far.\n\nI found them in only 10 files?\n\n    $ find ./designate -name \"*.py\" | xargs grep -il \u0027coding\u0027 | wc -l\n    10","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"4c442451ce04100e2f1ab38766abeecf9f97be73","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":".. code-block:: python"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"    # -*- coding: utf-8 -*-"},{"line_number":63,"context_line":"    # Copyright \u003cyear\u003e \u003ccompany\u003e"},{"line_number":64,"context_line":"    #"},{"line_number":65,"context_line":"    # Author: \u003cname\u003e \u003cemail addr\u003e"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a1b2928_b91ae515","line":62,"in_reply_to":"9a061dce_9681a6fe","updated":"2016-04-11 11:26:58.000000000","message":"Done","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":11662,"name":"Paul Glass","email":"paul.glass@rackspace.com","username":"pglass"},"change_message_id":"c25121fc2caed7fc649f77b5f51dab5ad0b94200","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        `Specs: FILL_THIS Session \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_"},{"line_number":87,"context_line":"    \"\"\"  # noqa"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Example:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_7945e315","line":87,"updated":"2016-04-08 16:37:02.000000000","message":"+1 for the explanatory comment.\n\nThe package.module, specs, and user docs lines are a bit crufty and I would rather leave them out, or mention that they are optional. They can get out of sync with reality over time.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"320b9316960943d9eafa44b54a2d0b966cd642a6","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        `Specs: FILL_THIS Session \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_"},{"line_number":87,"context_line":"    \"\"\"  # noqa"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Example:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_4f52fa35","line":87,"in_reply_to":"5a18252c_8ebfa1fa","updated":"2016-04-21 11:48:56.000000000","message":"I meant to use the \"User documentation\" link to point the reader to our documentation under doc/sources/*.rst - those filenames are pretty static and we can also reference by label.\n\nHaving a link to our documentation in the source code is a way to remind developers to update or improve the docs while working on the code.\n\nI can update the example to clarify.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"e06b23fefa05a17a10e51935c3cbd86fd5833858","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        `Specs: FILL_THIS Session \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_"},{"line_number":87,"context_line":"    \"\"\"  # noqa"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Example:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_90e6c1e6","line":87,"in_reply_to":"9a061dce_7945e315","updated":"2016-04-08 20:31:52.000000000","message":"agreed","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"cfcb970f301c3811dd46b31837410c4a07025bc4","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        `Specs: FILL_THIS Session \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_"},{"line_number":87,"context_line":"    \"\"\"  # noqa"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"Example:"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a18252c_8ebfa1fa","line":87,"in_reply_to":"9a061dce_90e6c1e6","updated":"2016-04-13 09:56:55.000000000","message":"I\u0027m not sure about these either. Especially anything with a link. Links change all the time...","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"e06b23fefa05a17a10e51935c3cbd86fd5833858","unresolved":false,"context_lines":[{"line_number":108,"context_line":".. code-block:: python"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    class MyClass(object):"},{"line_number":111,"context_line":"        \"\"\"\u003cdescription\u003e"},{"line_number":112,"context_line":"        mention a function :func:`foo` or a class :class:`Bar`"},{"line_number":113,"context_line":"        \"\"\""},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_d0f7998a","line":111,"range":{"start_line":111,"start_character":11,"end_line":111,"end_character":24},"updated":"2016-04-08 20:31:52.000000000","message":"I hate to be that guy, but I really dislike when the description starts on the same line as the \"\"\", it just looks so unclean and unnecessary to me.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"2b3387e7d4a86270fb3d338298154df2378f2351","unresolved":false,"context_lines":[{"line_number":108,"context_line":".. code-block:: python"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    class MyClass(object):"},{"line_number":111,"context_line":"        \"\"\"\u003cdescription\u003e"},{"line_number":112,"context_line":"        mention a function :func:`foo` or a class :class:`Bar`"},{"line_number":113,"context_line":"        \"\"\""},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a18252c_71869dba","line":111,"updated":"2016-04-12 13:11:18.000000000","message":"It is an openstack thing - we actually test for it in the gate. see http://docs.openstack.org/developer/hacking/#docstrings\n\n(I agree with you, but we have been enforcing this until now.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"320b9316960943d9eafa44b54a2d0b966cd642a6","unresolved":false,"context_lines":[{"line_number":108,"context_line":".. code-block:: python"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    class MyClass(object):"},{"line_number":111,"context_line":"        \"\"\"\u003cdescription\u003e"},{"line_number":112,"context_line":"        mention a function :func:`foo` or a class :class:`Bar`"},{"line_number":113,"context_line":"        \"\"\""},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_af1aae7a","line":111,"in_reply_to":"5a18252c_71869dba","updated":"2016-04-21 11:48:56.000000000","message":"Done","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"320b9316960943d9eafa44b54a2d0b966cd642a6","unresolved":false,"context_lines":[{"line_number":108,"context_line":".. code-block:: python"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    class MyClass(object):"},{"line_number":111,"context_line":"        \"\"\"\u003cdescription\u003e"},{"line_number":112,"context_line":"        mention a function :func:`foo` or a class :class:`Bar`"},{"line_number":113,"context_line":"        \"\"\""},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_4f295a9a","line":111,"range":{"start_line":111,"start_character":11,"end_line":111,"end_character":24},"in_reply_to":"9a061dce_d0f7998a","updated":"2016-04-21 11:48:56.000000000","message":"Done","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"e06b23fefa05a17a10e51935c3cbd86fd5833858","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    LOG.info(_LI(\"... %(key)s ...\"), {\u0027key\u0027: \u0027value\u0027, ...})"},{"line_number":146,"context_line":"    LOG.warn(_LW(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"},{"line_number":147,"context_line":"    LOG.error(_LE(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"},{"line_number":148,"context_line":"    LOG.critical(_LC(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a061dce_70396d41","line":148,"updated":"2016-04-08 20:31:52.000000000","message":"It makes me oh so sad that we have to do this, but I get it.","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"320b9316960943d9eafa44b54a2d0b966cd642a6","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    LOG.info(_LI(\"... %(key)s ...\"), {\u0027key\u0027: \u0027value\u0027, ...})"},{"line_number":146,"context_line":"    LOG.warn(_LW(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"},{"line_number":147,"context_line":"    LOG.error(_LE(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"},{"line_number":148,"context_line":"    LOG.critical(_LC(\"... %(key)s\"), {\u0027key\u0027: \u0027value\u0027})"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_0f23e2bd","line":148,"in_reply_to":"9a061dce_70396d41","updated":"2016-04-21 11:48:56.000000000","message":"Done","commit_id":"a2031cda2dd1ae3ac9a51c0f02451d2107a6a7d8"},{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"5f6871b84b808b0f138032095c34921ffed0d4f9","unresolved":false,"context_lines":[{"line_number":49,"context_line":"Style Guide"},{"line_number":50,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Follow `OpenStack Style Guidelines \u003chttp://docs.openstack.org/developer/hacking/\u003e`_ and, optionally,"},{"line_number":53,"context_line":"the `Pocoo Style Guide \u003chttp://www.pocoo.org/internal/styleguide/\u003e`_"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"File header"},{"line_number":56,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"dab17558_924b5f2b","line":53,"range":{"start_line":52,"start_character":84,"end_line":53,"end_character":68},"updated":"2016-05-06 12:58:03.000000000","message":"Most everyone agreed on PS#1 to not defer to an external styleguide.","commit_id":"11942791bf2fcc0e15d4635fcaca2d9682aed583"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"06acdfd9ed3e9e6b92deb3eac04abb06e76892c0","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    # License for the specific language governing permissions and limitations"},{"line_number":75,"context_line":"    # under the License."},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    \"\"\""},{"line_number":78,"context_line":"        \u003cpackage.module\u003e"},{"line_number":79,"context_line":"        ~~~~~~~~~~~~~~"},{"line_number":80,"context_line":"        \u003cDescribe what the module should do, especially interactions with other components and caveats\u003e"}],"source_content_type":"text/x-rst","patch_set":4,"id":"dab17558_a853986b","line":77,"updated":"2016-05-12 13:49:15.000000000","message":"The first line should be on the same line as \"\"\" - it is a hacking check","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"58c8c7db17e5abfa9f0cf88c3e1cd9df3f3de990","unresolved":false,"context_lines":[{"line_number":74,"context_line":"    # License for the specific language governing permissions and limitations"},{"line_number":75,"context_line":"    # under the License."},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    \"\"\""},{"line_number":78,"context_line":"        \u003cpackage.module\u003e"},{"line_number":79,"context_line":"        ~~~~~~~~~~~~~~"},{"line_number":80,"context_line":"        \u003cDescribe what the module should do, especially interactions with other components and caveats\u003e"}],"source_content_type":"text/x-rst","patch_set":4,"id":"dab17558_e46fc928","line":77,"in_reply_to":"dab17558_a853986b","updated":"2016-05-17 12:56:07.000000000","message":"Done","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"06acdfd9ed3e9e6b92deb3eac04abb06e76892c0","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        `Specs: FILL_THIS \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_ \u003cRefer to files under doc by path or by label\u003e"},{"line_number":86,"context_line":"    \"\"\"  # noqa"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Example:"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"dab17558_88505460","line":86,"updated":"2016-05-12 13:49:15.000000000","message":"If we have to add # noqa to the example we are doing it wrong","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"58c8c7db17e5abfa9f0cf88c3e1cd9df3f3de990","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        `Specs: FILL_THIS \u003chttps://github.com/openstack/designate-specs/blob/master/specs/FILL_THIS.rst\u003e`_"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        `User documentation \u003cFILL_THIS.html\u003e`_ \u003cRefer to files under doc by path or by label\u003e"},{"line_number":86,"context_line":"    \"\"\"  # noqa"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Example:"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"dab17558_8484fdd2","line":86,"in_reply_to":"dab17558_88505460","updated":"2016-05-17 12:56:07.000000000","message":"Done","commit_id":"83ed60109f8a88c5a70397ce89ac3ad17fd8b3f7"}]}