)]}'
{"doc/source/common/git.rst":[{"author":{"_account_id":21723,"name":"Gergely Csatari","email":"gergely.csatari@nokia.com","username":"csatari"},"change_message_id":"41dd4446f55c77395a740a3cc7b4a3f0acca4377","unresolved":false,"context_lines":[{"line_number":105,"context_line":"- Body"},{"line_number":106,"context_line":"- Tags"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Summary Line"},{"line_number":109,"context_line":"------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"The summary line briefly describes the patch content. The character limit is 50"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bf659307_e5bcfd6d","line":108,"range":{"start_line":108,"start_character":0,"end_line":108,"end_character":12},"updated":"2018-04-06 13:36:40.000000000","message":"Maybe we should say something about the WIP changes :)","commit_id":"d8722ec180ecbcf55410e0797e02b4ec776b4515"},{"author":{"_account_id":21723,"name":"Gergely Csatari","email":"gergely.csatari@nokia.com","username":"csatari"},"change_message_id":"41dd4446f55c77395a740a3cc7b4a3f0acca4377","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Body"},{"line_number":115,"context_line":"----"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The body contains the explanation of the issue being solved and why it should be"},{"line_number":118,"context_line":"fixed, the description of the solution, and additional optional information"},{"line_number":119,"context_line":"on how it improves the code structure, or references to other relevant patches,"},{"line_number":120,"context_line":"for example. The lines are limited to 72 characters. The body should contain all"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bf659307_65b0ed8c","line":117,"range":{"start_line":117,"start_character":78,"end_line":117,"end_character":80},"updated":"2018-04-06 13:36:40.000000000","message":"Line too long (Zuul also complains about this)","commit_id":"d8722ec180ecbcf55410e0797e02b4ec776b4515"},{"author":{"_account_id":21723,"name":"Gergely Csatari","email":"gergely.csatari@nokia.com","username":"csatari"},"change_message_id":"41dd4446f55c77395a740a3cc7b4a3f0acca4377","unresolved":false,"context_lines":[{"line_number":117,"context_line":"The body contains the explanation of the issue being solved and why it should be"},{"line_number":118,"context_line":"fixed, the description of the solution, and additional optional information"},{"line_number":119,"context_line":"on how it improves the code structure, or references to other relevant patches,"},{"line_number":120,"context_line":"for example. The lines are limited to 72 characters. The body should contain all"},{"line_number":121,"context_line":"the important information related to the problem, without assuming that the"},{"line_number":122,"context_line":"reader understands the source of the problem or has access to external sites."}],"source_content_type":"text/x-rst","patch_set":1,"id":"bf659307_85ad0133","line":120,"range":{"start_line":120,"start_character":77,"end_line":120,"end_character":80},"updated":"2018-04-06 13:36:40.000000000","message":"Line too long (Zuul also complains about this)","commit_id":"d8722ec180ecbcf55410e0797e02b4ec776b4515"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f27612e50615c599e8b295674fcb64a1dba955cc","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Format:"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"- Summary Line"},{"line_number":105,"context_line":"- Body"},{"line_number":106,"context_line":"- Tags"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Summary Line"},{"line_number":109,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_5c6b9bb5","line":106,"range":{"start_line":105,"start_character":0,"end_line":106,"end_character":6},"updated":"2018-04-13 12:26:45.000000000","message":"I suggest to add that there should be an empty line after the summary line.","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":22534,"name":"Andrea Visnyei","email":"andrea.visnyei@nokia.com","username":"visnyei"},"change_message_id":"c1caf0595a2edf391906e23fabfa17db43c5c304","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Format:"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"- Summary Line"},{"line_number":105,"context_line":"- Body"},{"line_number":106,"context_line":"- Tags"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Summary Line"},{"line_number":109,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9f6a8fd7_1c48115a","line":106,"range":{"start_line":105,"start_character":0,"end_line":106,"end_character":6},"in_reply_to":"bf659307_5c6b9bb5","updated":"2018-04-20 08:23:08.000000000","message":"Done","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f27612e50615c599e8b295674fcb64a1dba955cc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"  changed when rebasing a commit following review feedback, since it is used"},{"line_number":135,"context_line":"  by Gerrit, to track versions of a patch."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"The following tags are optional but recommended:"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"- ``Closes-Bug``: #123456789: use ``Closes-Bug`` if the commit is intended to"},{"line_number":140,"context_line":"  fully fix and close the bug being referenced. Use the Launchpad ID of the bug"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_fc886f71","line":137,"range":{"start_line":137,"start_character":32,"end_line":137,"end_character":48},"updated":"2018-04-13 12:26:45.000000000","message":"let\u0027s reword this. They are recommend to be used for these cases, so a patch writer should check which of these to add and then use them.\n\nMy first impression on your line is: now we have somebody submit each and every tag...","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":22534,"name":"Andrea Visnyei","email":"andrea.visnyei@nokia.com","username":"visnyei"},"change_message_id":"c1caf0595a2edf391906e23fabfa17db43c5c304","unresolved":false,"context_lines":[{"line_number":134,"context_line":"  changed when rebasing a commit following review feedback, since it is used"},{"line_number":135,"context_line":"  by Gerrit, to track versions of a patch."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"The following tags are optional but recommended:"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"- ``Closes-Bug``: #123456789: use ``Closes-Bug`` if the commit is intended to"},{"line_number":140,"context_line":"  fully fix and close the bug being referenced. Use the Launchpad ID of the bug"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9f6a8fd7_3c45d543","line":137,"range":{"start_line":137,"start_character":32,"end_line":137,"end_character":48},"in_reply_to":"bf659307_fc886f71","updated":"2018-04-20 08:23:08.000000000","message":"Done","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f27612e50615c599e8b295674fcb64a1dba955cc","unresolved":false,"context_lines":[{"line_number":154,"context_line":"  Use this tag to indicate that documentation is either contained in the"},{"line_number":155,"context_line":"  patch or has documentation impact."},{"line_number":156,"context_line":"  When this tag is included in a commit"},{"line_number":157,"context_line":"  message, Gerrit creates a bug for the openstack-manuals project for task"},{"line_number":158,"context_line":"  tracking, or move to the openstack-api-site as needed."},{"line_number":159,"context_line":"- The ``APIImpact`` tag contains a comment about why"},{"line_number":160,"context_line":"  the change impacts a public HTTP API. Put APIImpact on a line by itself."},{"line_number":161,"context_line":"  Use this tag to indicate that the patch impacts a public"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf659307_7c3d3fb9","line":158,"range":{"start_line":157,"start_character":0,"end_line":158,"end_character":56},"updated":"2018-04-13 12:26:45.000000000","message":"this is not correct anymore - the bug is filed against the project itself, see https://docs.openstack.org/doc-contrib-guide/doc-impact.html","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":22534,"name":"Andrea Visnyei","email":"andrea.visnyei@nokia.com","username":"visnyei"},"change_message_id":"c1caf0595a2edf391906e23fabfa17db43c5c304","unresolved":false,"context_lines":[{"line_number":154,"context_line":"  Use this tag to indicate that documentation is either contained in the"},{"line_number":155,"context_line":"  patch or has documentation impact."},{"line_number":156,"context_line":"  When this tag is included in a commit"},{"line_number":157,"context_line":"  message, Gerrit creates a bug for the openstack-manuals project for task"},{"line_number":158,"context_line":"  tracking, or move to the openstack-api-site as needed."},{"line_number":159,"context_line":"- The ``APIImpact`` tag contains a comment about why"},{"line_number":160,"context_line":"  the change impacts a public HTTP API. Put APIImpact on a line by itself."},{"line_number":161,"context_line":"  Use this tag to indicate that the patch impacts a public"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9f6a8fd7_dc77f99f","line":158,"range":{"start_line":157,"start_character":0,"end_line":158,"end_character":56},"in_reply_to":"bf659307_7c3d3fb9","updated":"2018-04-20 08:23:08.000000000","message":"Done","commit_id":"62b51fda65e0d0930ee3315fb6e20d38172bd9fe"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"0875001432b4edbb7f261e7d32dcd88e30b58730","unresolved":false,"context_lines":[{"line_number":109,"context_line":"Summary Line"},{"line_number":110,"context_line":"------------"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"The summary line briefly describes the patch content. The character limit is 50"},{"line_number":113,"context_line":"characters. The summary line should not end with a period. If the change is"},{"line_number":114,"context_line":"not finished at the time of the commit, start the commit message with WIP."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Body"},{"line_number":117,"context_line":"----"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9f6a8fd7_fbcdfe84","line":114,"range":{"start_line":112,"start_character":54,"end_line":114,"end_character":74},"updated":"2018-04-23 09:44:49.000000000","message":"This could be a list of things to keep in mind.","commit_id":"0b14b1aca024c473f1474aa279ea6a488899ed6e"}]}