)]}'
{"doc/contributor-guide/source/docs-review.rst":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":20,"context_line":"special rules apply:"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"* api projects: both the OpenStack project core and the Docs core are cores."},{"line_number":23,"context_line":"  For image-api the keystone cores are cores as well."},{"line_number":24,"context_line":"* docs-specs: has a separate core team, see docs-specs section."},{"line_number":25,"context_line":"* security-doc: has a separate core team consisting of Docs team members and"},{"line_number":26,"context_line":"  Security team members. The rule here is that each patch needs an approval"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_d7069acd","line":23,"updated":"2015-10-07 14:07:36.000000000","message":"Not true anymore, we only have *one* api project, the api-site repository. It shares the cores with openstack-manuals.","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":20,"context_line":"special rules apply:"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"* api projects: both the OpenStack project core and the Docs core are cores."},{"line_number":23,"context_line":"  For image-api the keystone cores are cores as well."},{"line_number":24,"context_line":"* docs-specs: has a separate core team, see docs-specs section."},{"line_number":25,"context_line":"* security-doc: has a separate core team consisting of Docs team members and"},{"line_number":26,"context_line":"  Security team members. The rule here is that each patch needs an approval"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_62b7bf86","line":23,"in_reply_to":"7a2fa921_d7069acd","updated":"2015-10-07 16:33:41.000000000","message":"Removed this item.","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":25,"context_line":"* security-doc: has a separate core team consisting of Docs team members and"},{"line_number":26,"context_line":"  Security team members. The rule here is that each patch needs an approval"},{"line_number":27,"context_line":"  by a Docs core and a Security core."},{"line_number":28,"context_line":"* training-guides: has a separate core team."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":".. TODO: add a link to docs-specs section once converted to RST:"},{"line_number":31,"context_line":"   https://wiki.openstack.org/wiki/Documentation/HowTo#When_to_write_a_blueprint_description_in_docs-specs"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_f73076a0","line":28,"updated":"2015-10-07 14:07:36.000000000","message":"Add training-labs, has also separate core team","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":25,"context_line":"* security-doc: has a separate core team consisting of Docs team members and"},{"line_number":26,"context_line":"  Security team members. The rule here is that each patch needs an approval"},{"line_number":27,"context_line":"  by a Docs core and a Security core."},{"line_number":28,"context_line":"* training-guides: has a separate core team."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":".. TODO: add a link to docs-specs section once converted to RST:"},{"line_number":31,"context_line":"   https://wiki.openstack.org/wiki/Documentation/HowTo#When_to_write_a_blueprint_description_in_docs-specs"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_c24ed395","line":28,"in_reply_to":"7a2fa921_f73076a0","updated":"2015-10-07 16:33:41.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":58,"context_line":"* Now, the existing core team can act faster on recognizing valuable team"},{"line_number":59,"context_line":"  members."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The new process is:"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"- Every month (usually on the 1st), the documentation PTL draws the top 12"},{"line_number":64,"context_line":"  names using these reports:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_3717ceee","line":61,"updated":"2015-10-07 14:07:36.000000000","message":"Remove \"new\"","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":58,"context_line":"* Now, the existing core team can act faster on recognizing valuable team"},{"line_number":59,"context_line":"  members."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The new process is:"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"- Every month (usually on the 1st), the documentation PTL draws the top 12"},{"line_number":64,"context_line":"  names using these reports:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_8248db88","line":61,"in_reply_to":"7a2fa921_3717ceee","updated":"2015-10-07 16:33:41.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":79,"context_line":"How to review a documentation patch"},{"line_number":80,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Before you proceed with reviewing patches, make sure to read carefully the"},{"line_number":83,"context_line":"`Review Guidelines`_ for documentation. Once done, follow the steps below"},{"line_number":84,"context_line":"to submit a patch review."},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_d7bbfae3","line":82,"updated":"2015-10-07 14:07:36.000000000","message":"add a reference to the infra manual: http://docs.openstack.org/infra/manual/developers.html#code-review","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":79,"context_line":"How to review a documentation patch"},{"line_number":80,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"Before you proceed with reviewing patches, make sure to read carefully the"},{"line_number":83,"context_line":"`Review Guidelines`_ for documentation. Once done, follow the steps below"},{"line_number":84,"context_line":"to submit a patch review."},{"line_number":85,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_7d6f64c5","line":82,"in_reply_to":"7a2fa921_d7bbfae3","updated":"2015-10-07 16:33:41.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"   For books in .RST format, run::"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"     sudo tox -e checkbuild"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"   For books in .xml format:"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_57412aa8","line":137,"updated":"2015-10-07 14:07:36.000000000","message":"This works for *all* formats","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"   For books in .RST format, run::"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"     sudo tox -e checkbuild"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"   For books in .xml format:"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_0e9074eb","line":137,"in_reply_to":"7a2fa921_57412aa8","updated":"2015-10-07 16:33:41.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":150,"context_line":""},{"line_number":151,"context_line":"#. Check the build result:"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"   * For books in .RST format, find the :file:`build` folder of a guide"},{"line_number":154,"context_line":"     you review a patch for. For example,"},{"line_number":155,"context_line":"     openstack-manuals/doc/admin-guide-cloud/build/html."},{"line_number":156,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_77230606","line":153,"updated":"2015-10-07 14:07:36.000000000","message":"published-docs contains always the content, we can condense the section.","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":150,"context_line":""},{"line_number":151,"context_line":"#. Check the build result:"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"   * For books in .RST format, find the :file:`build` folder of a guide"},{"line_number":154,"context_line":"     you review a patch for. For example,"},{"line_number":155,"context_line":"     openstack-manuals/doc/admin-guide-cloud/build/html."},{"line_number":156,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_4ead0c30","line":153,"in_reply_to":"7a2fa921_77230606","updated":"2015-10-07 16:33:41.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"26a8f8c94ac295ad3302c4832bc4559e453ede09","unresolved":false,"context_lines":[{"line_number":171,"context_line":"   #. When the editor opens, update the commit message if necessary. But do"},{"line_number":172,"context_line":"      not add information on what your specific patch set changes. A reviewer"},{"line_number":173,"context_line":"      can use the Gerrit interface to see the difference between patches."},{"line_number":174,"context_line":"   #. Enter colon + w + q to save and exit the editor."},{"line_number":175,"context_line":"   #. Send your patch to the existing review::"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"       git review"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_f7995620","line":174,"updated":"2015-10-07 14:07:36.000000000","message":"This is \"vi\" specific - we should mention that","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"07c8bbe4c1e57ef8a76909b9fb4c2f0edb42bd97","unresolved":false,"context_lines":[{"line_number":171,"context_line":"   #. When the editor opens, update the commit message if necessary. But do"},{"line_number":172,"context_line":"      not add information on what your specific patch set changes. A reviewer"},{"line_number":173,"context_line":"      can use the Gerrit interface to see the difference between patches."},{"line_number":174,"context_line":"   #. Enter colon + w + q to save and exit the editor."},{"line_number":175,"context_line":"   #. Send your patch to the existing review::"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"       git review"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_7313100e","line":174,"in_reply_to":"7a2fa921_33159b3f","updated":"2015-10-07 16:39:53.000000000","message":"This was written in this way since vi is default for many. So, what combining \"Save the changes and exit the editor. If your editors is vi, enter a colon (\":\")... to save the file and exit vi.\"","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"96b449fcdc929234dde76cd7378fb947cf7803ae","unresolved":false,"context_lines":[{"line_number":171,"context_line":"   #. When the editor opens, update the commit message if necessary. But do"},{"line_number":172,"context_line":"      not add information on what your specific patch set changes. A reviewer"},{"line_number":173,"context_line":"      can use the Gerrit interface to see the difference between patches."},{"line_number":174,"context_line":"   #. Enter colon + w + q to save and exit the editor."},{"line_number":175,"context_line":"   #. Send your patch to the existing review::"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"       git review"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_d7abd073","line":174,"in_reply_to":"7a2fa921_7313100e","updated":"2015-10-08 09:53:31.000000000","message":"Done","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"f9856b5379ce26242279cfc708ef7a7370689f8e","unresolved":false,"context_lines":[{"line_number":171,"context_line":"   #. When the editor opens, update the commit message if necessary. But do"},{"line_number":172,"context_line":"      not add information on what your specific patch set changes. A reviewer"},{"line_number":173,"context_line":"      can use the Gerrit interface to see the difference between patches."},{"line_number":174,"context_line":"   #. Enter colon + w + q to save and exit the editor."},{"line_number":175,"context_line":"   #. Send your patch to the existing review::"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"       git review"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7a2fa921_33159b3f","line":174,"in_reply_to":"7a2fa921_f7995620","updated":"2015-10-07 16:33:41.000000000","message":"Is it okay to make this sentence more general to cover any editor a contributor may have? -\u003e Save the changes if any and exit the editor.","commit_id":"aac342c6dfa402ef825bd34275713ec2fbc12107"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"988dbea35f7d8146ae49103414266e78cc725b91","unresolved":false,"context_lines":[{"line_number":14,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The OpenStack Docs team is core for api-site, openstack-manuals,"},{"line_number":17,"context_line":"operations-guide and openstack-doc-tools projects."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"For the following repositories that are part of the Documentation program,"},{"line_number":20,"context_line":"special rules apply:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a2fa921_fd158de4","line":17,"updated":"2015-10-08 10:30:07.000000000","message":"\",\" before and\n\nAdd to the list: openstackdocstheme","commit_id":"9fafe4694b6a5776b42e33bf61220c1dfc93c3d5"},{"author":{"_account_id":14962,"name":"ologvinova","email":"ologvinova@mirantis.com","username":"ologvinova"},"change_message_id":"eb1981a88188ea37ba69044a05b5f36b9b3ca387","unresolved":false,"context_lines":[{"line_number":14,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"The OpenStack Docs team is core for api-site, openstack-manuals,"},{"line_number":17,"context_line":"operations-guide and openstack-doc-tools projects."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"For the following repositories that are part of the Documentation program,"},{"line_number":20,"context_line":"special rules apply:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a2fa921_9d06d984","line":17,"in_reply_to":"7a2fa921_fd158de4","updated":"2015-10-08 10:39:12.000000000","message":"Done","commit_id":"9fafe4694b6a5776b42e33bf61220c1dfc93c3d5"}]}