)]}'
{"api-ref/source/parameters.yaml":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"31c23ea01d85a559adaf2164821da9e14bae2cab","unresolved":false,"context_lines":[{"line_number":196,"context_line":"  in: header"},{"line_number":197,"context_line":"  required: false"},{"line_number":198,"context_line":"  type: string"},{"line_number":199,"context_line":"If-None-Match-get-request:"},{"line_number":200,"context_line":"  description: |"},{"line_number":201,"context_line":"    A client that has one or more entities previously"},{"line_number":202,"context_line":"    obtained from the resource can verify that none of those entities is"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_a2096586","line":199,"range":{"start_line":199,"start_character":18,"end_line":199,"end_character":25},"updated":"2016-12-12 04:12:56.000000000","message":"Miner nits)\n\nIt looks like other tests using under score and `req` for request description. i.e. If-None-Match_get_req for this and If-None-Match_put_req for the other.","commit_id":"07d3dd7c7f2d2b1a849b028bc6bfe5c422cccbe5"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"31c23ea01d85a559adaf2164821da9e14bae2cab","unresolved":false,"context_lines":[{"line_number":198,"context_line":"  type: string"},{"line_number":199,"context_line":"If-None-Match-get-request:"},{"line_number":200,"context_line":"  description: |"},{"line_number":201,"context_line":"    A client that has one or more entities previously"},{"line_number":202,"context_line":"    obtained from the resource can verify that none of those entities is"},{"line_number":203,"context_line":"    current by including a list of their associated entity tags in the"},{"line_number":204,"context_line":"    ``If-None-Match header`` field."},{"line_number":205,"context_line":"    See `Request for Comments: 2616 \u003chttp://www.ietf.org/rfc/rfc2616.txt\u003e`_"},{"line_number":206,"context_line":"    for details."},{"line_number":207,"context_line":"  in: header"},{"line_number":208,"context_line":"  required: false"},{"line_number":209,"context_line":"  type: string"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a71b18c_ce9a0b14","line":206,"range":{"start_line":201,"start_character":3,"end_line":206,"end_character":15},"updated":"2016-12-12 04:12:56.000000000","message":"This looks great rather than just \u0027See \u003clink\u003e\u0027. Nice.","commit_id":"07d3dd7c7f2d2b1a849b028bc6bfe5c422cccbe5"}],"api-ref/source/storage-object-services.inc":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"bbbc36a0a008cb082d75293eced3d65331849fbf","unresolved":false,"context_lines":[{"line_number":554,"context_line":"   - multipart-manifest: multipart-manifest_head"},{"line_number":555,"context_line":"   - X-Newest: X-Newest"},{"line_number":556,"context_line":"   - If-Match: If-Match"},{"line_number":557,"context_line":"   - If-None-Match: If-None-Match"},{"line_number":558,"context_line":"   - If-Modified-Since: If-Modified-Since"},{"line_number":559,"context_line":"   - If-Unmodified-Since: If-Unmodified-Since"},{"line_number":560,"context_line":"   - X-Trans-Id-Extra: X-Trans-Id-Extra"}],"source_content_type":"text/x-c++src","patch_set":1,"id":"3a71b18c_f6c43193","line":557,"range":{"start_line":557,"start_character":1,"end_line":557,"end_character":33},"updated":"2016-12-09 12:27:11.000000000","message":"Looking at the api reference built with this change, the description looks to result into odd sentence.\n\nI found:\n\nIn combination with Expect: 100-Continue, specify an \"If-None-Match: *\" header to query whether the server already has a copy of the object before any data is sent.\n\nfor If-None-Match.\n\nOthers are:\n\nSee Request for Comments: 2616.\n\nThis looks ok.\n\nAnd I found the odd sentences appears also for GET request description so the problem is not related to whole of this change but partially.\n\nThe reason for the odd sentence would be probably we\u0027re using same description for PUT and GET/HEAD. That description (i.e. Expect header and If-Non-Match combination) is for PUT object and the behavior is not for GET/HEAD.\n\nI think, the better way is making separate description like If-None-Match-PUT and If-None-Match-GETHEAD and then set for each item to suitable APIs.","commit_id":"aa2b6a7818faabbcd2f4839e0d45ff6bf7f13719"}]}
