)]}'
{"doc/source/contributor/code-reviews.rst":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"54b8df2b1bedee90147cb8b8df25afcec55a244d","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"You can watch specific patches in Gerrit via *Settings -\u003e Watched Projects*."},{"line_number":17,"context_line":"The volume of emails is not too large if you subscribe to *New Changes* only."},{"line_number":18,"context_line":"If you do not have much time available for reviewng, consider reviewing patches"},{"line_number":19,"context_line":"in an area that is important to you or that you understand well."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Core reviewers"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f621f24_1edd6ddf","line":18,"range":{"start_line":18,"start_character":43,"end_line":18,"end_character":51},"updated":"2020-11-16 11:57:14.000000000","message":"nit: s/reviewng/reviewing","commit_id":"f32b292c1babdc237836d3b43b723e532014bdd2"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"eda215e2abf2465b2bbdb13e46a439f30ef94cad","unresolved":false,"context_lines":[{"line_number":6,"context_line":"with a Gerrit account is able to provide a review. Two labels are available to"},{"line_number":7,"context_line":"everyone:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"+1: Approve"},{"line_number":10,"context_line":"-1: Changes requested"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"It is also possible to leave comments without a label. In general, a review"},{"line_number":13,"context_line":"with comments is more valuable. Comments are especially important for a"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1f621f24_835ab37f","line":10,"range":{"start_line":9,"start_character":4,"end_line":10,"end_character":11},"updated":"2020-11-17 15:37:07.000000000","message":"these...","commit_id":"9308847d094a640072e3b336ddbaee7c0db1c877"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"eda215e2abf2465b2bbdb13e46a439f30ef94cad","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Core reviewers have additional labels available to them."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"+2: Approve"},{"line_number":27,"context_line":"-2: Do not merge"},{"line_number":28,"context_line":"Workflow +1: Approve and ready for merge"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Zuul requires one +2 and one workflow +1, as well as a passing check, in order"},{"line_number":31,"context_line":"for a patch to proceed to the gate. The Kolla team generally requires two +2s"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1f621f24_034ea3bf","line":28,"range":{"start_line":26,"start_character":4,"end_line":28,"end_character":12},"updated":"2020-11-17 15:37:07.000000000","message":"... and these render bad, I\u0027d expect a list perhaps?","commit_id":"9308847d094a640072e3b336ddbaee7c0db1c877"}]}