)]}'
{"REVIEWING.rst":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"3ae906821303534029e3a82c1c36a9d8389ef3ff","unresolved":false,"context_lines":[{"line_number":44,"context_line":"-------------"},{"line_number":45,"context_line":"Tests should only be added for a published stable APIs. If a patch contains"},{"line_number":46,"context_line":"tests for an API which hasn\u0027t been marked as stable in respective project,"},{"line_number":47,"context_line":"then it should not be approved."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Changes breaking API Interoperability"},{"line_number":50,"context_line":"-------------------------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3f4b6375_059c250d","line":47,"updated":"2017-10-24 07:58:08.000000000","message":"The change to the sentence here doesn\u0027t read clearly to me. I\u0027m not sure what it means anymore.","commit_id":"71e34ea164a34fb7314da8132cbbd218cd6e5afe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"31e3d418b9d4d56b890b2ce083215e74e73c0b35","unresolved":false,"context_lines":[{"line_number":44,"context_line":"-------------"},{"line_number":45,"context_line":"Tests should only be added for a published stable APIs. If a patch contains"},{"line_number":46,"context_line":"tests for an API which hasn\u0027t been marked as stable in respective project,"},{"line_number":47,"context_line":"then it should not be approved."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Changes breaking API Interoperability"},{"line_number":50,"context_line":"-------------------------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3f4b6375_202a6f7e","line":47,"in_reply_to":"3f4b6375_059c250d","updated":"2017-10-24 08:25:38.000000000","message":"I thought of separating the non stable API tests like test for experimental APIs if any.","commit_id":"71e34ea164a34fb7314da8132cbbd218cd6e5afe"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0af6750a4b228942f73e9b06a43ead45ee9fba17","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Changes breaking API Interoperability"},{"line_number":50,"context_line":"-------------------------------------"},{"line_number":51,"context_line":"Along with integration testing, Tempest validate the API Interoperability also."},{"line_number":52,"context_line":"API Interoperability guidelines are defined in `API Interoperability guidelines"},{"line_number":53,"context_line":"\u003chttps://specs.openstack.org/openstack/api-wg/guidelines/api_interoperability.html\u003e`_"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3f4b6375_420b177c","line":51,"range":{"start_line":51,"start_character":40,"end_line":51,"end_character":48},"updated":"2017-10-25 16:55:59.000000000","message":"nit: s/validate/validates/","commit_id":"71e34ea164a34fb7314da8132cbbd218cd6e5afe"}]}