)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15640,"name":"Roxana Gherle","email":"roxana.gherle@hpe.com","username":"roxana-gherle"},"change_message_id":"2d4b07ac5918c5f538e9fc444ad91451f86c1148","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Error link is updated"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"extra http is removed"},{"line_number":10,"context_line":"underline for heading2 should be with \u0027~\u0027."},{"line_number":11,"context_line":"So it is modified"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da20952f_5f77959f","line":10,"updated":"2015-08-31 16:58:24.000000000","message":"Not a big thing, but could you reformat this description to either be a list starting with dash (-) of the things you fixed or a sentence starting with capital letter.","commit_id":"7fabe728afbc9cb5b4b4a15810c64f7c78fd5969"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"ae4de228e439fbc176fe3179faa59aa2a9234a44","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed and underline for description"},{"line_number":11,"context_line":"and mapping schema is changed"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba15a1d1_daee01ec","line":10,"updated":"2015-09-14 13:51:24.000000000","message":"I think its better to just split these two changes into separate patches.  That is if the underline change is even needed.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ec4d90c16410e28ce2a7a7f29d7651e38ebf45b9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed and underline for description"},{"line_number":11,"context_line":"and mapping schema is changed"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"},{"line_number":14,"context_line":"Closes-Bug: #1488903"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba15a1d1_ee5ec075","line":11,"updated":"2015-09-13 23:42:10.000000000","message":"the commit message should say why the change is being made. I can\u0027t see any difference in the output with the change to the underline for description and mapping schema, and nothing in the rst docs that says the old style was incorrect.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"},{"author":{"_account_id":17123,"name":"Tom Cocozzello (tjcocozz)","email":"tjcocozz@us.ibm.com","username":"tcocozzello"},"change_message_id":"2822c983214803176c83a21d98d4dfa94d29f343","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_ef045716","line":9,"updated":"2015-09-14 19:38:52.000000000","message":"Since the commit message is used as future reference after the code has been merged i think this should be changed.  Technically its \u0027http//\u0027","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"ae8a91c1a4c1b00f32eac60d94eaeba7430d2b77","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_c52cd9ad","line":9,"updated":"2015-09-14 17:57:21.000000000","message":"Technically its \"http://\" that\u0027s being removed, but not a big deal.\n\ns/Json/JSON","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"f530af2a01130d74dadfb9f6d1e63bddaf74b0e2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_d6031141","line":9,"in_reply_to":"ba15a1d1_8ff35b0a","updated":"2015-09-15 10:59:39.000000000","message":"I recommend to change this line to \"JSON schema link has redundant prefix\". You can avoid the problem that way.","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":17277,"name":"Christopher J Schaefer","email":"cjschaef@us.ibm.com","username":"cjschaef"},"change_message_id":"fec47abf9576c6ed8e8e77b76fc2eadbb385ffa7","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_8ff35b0a","line":9,"in_reply_to":"ba15a1d1_ef045716","updated":"2015-09-14 19:48:14.000000000","message":"It is \"http://\" in the doc, it just gets parsed and shows up as \"http//\" as part of the URL. If we are all being technical.","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"ae8a91c1a4c1b00f32eac60d94eaeba7430d2b77","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"},{"line_number":13,"context_line":"Closes-Bug: #1488903"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_851d8175","line":10,"updated":"2015-09-14 17:57:21.000000000","message":"it has been removed.","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":17277,"name":"Christopher J Schaefer","email":"cjschaef@us.ibm.com","username":"cjschaef"},"change_message_id":"fec47abf9576c6ed8e8e77b76fc2eadbb385ffa7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix the http link for JSON schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Json schema link has extra \"http/\" so"},{"line_number":10,"context_line":"it is removed"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"},{"line_number":13,"context_line":"Closes-Bug: #1488903"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba15a1d1_8f583bfc","line":10,"in_reply_to":"ba15a1d1_851d8175","updated":"2015-09-14 19:48:14.000000000","message":"I would recommend something along these lines:\n\n\"The Json schema link has an extra \"http://\", which should be removed for the link to function properly.\"","commit_id":"af36918ec256ed6a183412ffa555b5d61a075a16"},{"author":{"_account_id":17277,"name":"Christopher J Schaefer","email":"cjschaef@us.ibm.com","username":"cjschaef"},"change_message_id":"7e38b5916873864d47b6eeb3c7df3214326091bf","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The redirect link given for Json "},{"line_number":10,"context_line":"schema has extra \"http://\"."},{"line_number":11,"context_line":"It is removed from the link to work it"},{"line_number":12,"context_line":"properly"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I4da9c9301db055a62679c4ca1da73ae728220e88"},{"line_number":15,"context_line":"Closes-Bug: #1488903"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba15a1d1_b180ce47","line":12,"updated":"2015-09-15 19:08:28.000000000","message":"This still isn\u0027t grammatically correct. Please fix this, such as:\n\n\"The redirect link provided for the Json schema has an extra \"http://\" embedded within the link. This has been removed to allow the link to function properly.\"","commit_id":"d54b262ceabe29e6d494661fb5d5b107fb146f21"}],"doc/source/mapping_schema.rst":[{"author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"change_message_id":"0fa6fd0d9ce73a4131ae2370fe5a2a54aeb1018d","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Description"},{"line_number":19,"context_line":"~~~~~~~~~~~"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The schema for mapping is a description of how a mapping should be created."},{"line_number":22,"context_line":"It shows all the requirements and possibilities for a JSON to be used for mapping."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da20952f_7325c0c0","line":19,"updated":"2015-08-26 17:38:35.000000000","message":"Why you\u0027re changing this?\n\nAs I know, In every rst documentation here, we use \u0027-------\u0027 as the partner for titles.","commit_id":"7fabe728afbc9cb5b4b4a15810c64f7c78fd5969"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a84be08a6039734ef73c89442e046695e499a0c4","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Description"},{"line_number":19,"context_line":"~~~~~~~~~~~"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The schema for mapping is a description of how a mapping should be created."},{"line_number":22,"context_line":"It shows all the requirements and possibilities for a JSON to be used for mapping."}],"source_content_type":"text/x-rst","patch_set":1,"id":"da20952f_5ae7b1ab","line":19,"in_reply_to":"da20952f_7325c0c0","updated":"2015-08-31 13:36:28.000000000","message":"We should do this in a separate commit. I don\u0027t think there is a reason to change the heading types, is there? \n\nI think the bug linked to this review only mentions the broken link: \n\nhttps://bugs.launchpad.net/keystone/+bug/1488903","commit_id":"7fabe728afbc9cb5b4b4a15810c64f7c78fd5969"},{"author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"change_message_id":"bf15e929d2fa04b356fe98fc0da97870eb4cb52d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003chttp://json-schema.org/documentation.html\u003e`__"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Mapping Schema"},{"line_number":28,"context_line":"--------------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The rules supported must use the following schema:"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da20952f_b7168453","line":28,"updated":"2015-08-28 08:41:45.000000000","message":"why not changing this then?","commit_id":"7fabe728afbc9cb5b4b4a15810c64f7c78fd5969"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"c96a650bf71c28bd6a65e90dbf3f3aec41940c29","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003chttp://json-schema.org/documentation.html\u003e`__"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Mapping Schema"},{"line_number":28,"context_line":"--------------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The rules supported must use the following schema:"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"da20952f_972fadf5","line":28,"in_reply_to":"da20952f_b7168453","updated":"2015-08-28 14:48:18.000000000","message":"+1","commit_id":"7fabe728afbc9cb5b4b4a15810c64f7c78fd5969"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ec4d90c16410e28ce2a7a7f29d7651e38ebf45b9","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Description"},{"line_number":19,"context_line":"~~~~~~~~~~~"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The schema for mapping is a description of how a mapping should be created."},{"line_number":22,"context_line":"It shows all the requirements and possibilities for a JSON to be used for mapping."}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba15a1d1_8e54e454","line":19,"updated":"2015-09-13 23:42:10.000000000","message":"revert this change.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6991c4c8f5b072d2666e7717c9b17ead9da9bdaf","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Description"},{"line_number":19,"context_line":"~~~~~~~~~~~"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The schema for mapping is a description of how a mapping should be created."},{"line_number":22,"context_line":"It shows all the requirements and possibilities for a JSON to be used for mapping."}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba15a1d1_d731faed","line":19,"in_reply_to":"ba15a1d1_8e54e454","updated":"2015-09-14 13:23:40.000000000","message":"++, I think this is an unrelated change as it isn\u0027t detailed in the commit message.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ec4d90c16410e28ce2a7a7f29d7651e38ebf45b9","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003chttp://json-schema.org/documentation.html\u003e`__"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Mapping Schema"},{"line_number":28,"context_line":"~~~~~~~~~~~~~~"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The rules supported must use the following schema:"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba15a1d1_ae51a864","line":28,"updated":"2015-09-13 23:42:10.000000000","message":"revert this change.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6991c4c8f5b072d2666e7717c9b17ead9da9bdaf","unresolved":false,"context_lines":[{"line_number":25,"context_line":"\u003chttp://json-schema.org/documentation.html\u003e`__"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Mapping Schema"},{"line_number":28,"context_line":"~~~~~~~~~~~~~~"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"The rules supported must use the following schema:"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba15a1d1_37571604","line":28,"in_reply_to":"ba15a1d1_ae51a864","updated":"2015-09-14 13:23:40.000000000","message":"same comment as above.","commit_id":"a53d5e6e131a4c4e61285ad26dbaea0e6f122fc3"}]}