)]}'
{"doc/source/reviewers_guide.rst":[{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"9b6a921f119638a68f80125f1c0e99c29ca076ec","unresolved":false,"context_lines":[{"line_number":71,"context_line":"Dev-refs are usually reviewed and accepted as part of the implementation or"},{"line_number":72,"context_line":"implementation change."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Bugs \u0026 Blueprints"},{"line_number":75,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"For any issue with existing implementation, a bug report is expected."}],"source_content_type":"text/x-rst","patch_set":4,"id":"3f4b6375_1df66090","line":74,"updated":"2017-11-07 10:03:55.000000000","message":"blue-prints not mentioned in the paragraph below ..","commit_id":"55ecf30cc856c2a241b76ff57014b5f848a92a68"},{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"638cc937274979bd387f9e3002ced51cb1bd7c97","unresolved":false,"context_lines":[{"line_number":71,"context_line":"Dev-refs are usually reviewed and accepted as part of the implementation or"},{"line_number":72,"context_line":"implementation change."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Bugs \u0026 Blueprints"},{"line_number":75,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"For any issue with existing implementation, a bug report is expected."}],"source_content_type":"text/x-rst","patch_set":4,"id":"3f4b6375_6776cc0f","line":74,"in_reply_to":"3f4b6375_1df66090","updated":"2017-11-08 10:06:22.000000000","message":"Omer suggests using bugs system for both bugs and new features (as bugs tagged with special tag), to allow unified view of both.","commit_id":"55ecf30cc856c2a241b76ff57014b5f848a92a68"},{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"fcb1f421d897cb741f5c866b05268dbab1657697","unresolved":false,"context_lines":[{"line_number":71,"context_line":"Dev-refs are usually reviewed and accepted as part of the implementation or"},{"line_number":72,"context_line":"implementation change."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Bugs \u0026 Blueprints"},{"line_number":75,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"For any issue with existing implementation, a bug report is expected."}],"source_content_type":"text/x-rst","patch_set":4,"id":"3f4b6375_2a1f7b5c","line":74,"in_reply_to":"3f4b6375_6776cc0f","updated":"2017-11-08 10:25:00.000000000","message":"yep , i know . but i think that good-idea that will be inside the spec as other people could know as well..","commit_id":"55ecf30cc856c2a241b76ff57014b5f848a92a68"},{"author":{"_account_id":6598,"name":"Berezovsky Irena","email":"irenab.dev@gmail.com","username":"irenab"},"change_message_id":"a6261037e86f680e302504a2e256cfd16a20aa68","unresolved":false,"context_lines":[{"line_number":15,"context_line":"  A title explaining what is done. The body of the commit message should"},{"line_number":16,"context_line":"  concisely explain *what this change does* (if not trivial and covered by"},{"line_number":17,"context_line":"  the title) and *why this change is done* (if not obvious). Triviality and"},{"line_number":18,"context_line":"  obviousness are left to the reviewer\u0027s discretion"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# Tests:"},{"line_number":21,"context_line":"  Every change must be covered by tests. Unit tests are often the bare"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1f485f77_62d37c05","line":18,"range":{"start_line":18,"start_character":41,"end_line":18,"end_character":51},"updated":"2017-11-19 15:32:26.000000000","message":"missing \u0027.\u0027","commit_id":"bce92da537a173ea68920da17b9f042e17b7ae19"},{"author":{"_account_id":6598,"name":"Berezovsky Irena","email":"irenab.dev@gmail.com","username":"irenab"},"change_message_id":"a6261037e86f680e302504a2e256cfd16a20aa68","unresolved":false,"context_lines":[{"line_number":99,"context_line":"* One issue per bug"},{"line_number":100,"context_line":"  We are not affraid of bug reports. And they are easier to manage if each bug"},{"line_number":101,"context_line":"  is a single atomic issue we need to fix. (There are some exceptions to this"},{"line_number":102,"context_line":"  guideline, but they are usually very rare)"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1f485f77_02d4c8e7","line":102,"range":{"start_line":102,"start_character":43,"end_line":102,"end_character":44},"updated":"2017-11-19 15:32:26.000000000","message":"missing \u0027.\u0027","commit_id":"bce92da537a173ea68920da17b9f042e17b7ae19"}]}