)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"9149ba472de46dacd98949a5068ff433df4dc309","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[Optimize] Ensure decorator\u0027s required parameters exists"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add new decorator \u0027expects_func_args\u0027 to ensure custom"},{"line_number":10,"context_line":"decorator\u0027s required parameters exists."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I9f2bf77aa1784cfd6e85fca24bd126266e9e6458"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bf659307_233db18e","line":10,"updated":"2018-04-11 08:12:33.000000000","message":"-1: Could you elaborate a little bit on the benefits of this path?  As I don\u0027t see any real ones, in my eyes it would only be useful if programmers of new code don\u0027t pay attention, if there are no unit tests, no integration tests, and reviewers don\u0027t pay attention.\n\nPS: If this code comes from Nova it should be mentioned in the commit message.","commit_id":"0e67df9629039d2d4999125645bb5b6df8ef038f"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"3c86ea1c740965db59b0b58de690368589ac459d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     TommyLike \u003ctommylikehu@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-04-11 15:18:41 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Optimize] Ensure decorator\u0027s required parameters exist"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Sometimes the custom decorator would expect some parameters exist"},{"line_number":10,"context_line":"for usage, but it\u0027s possible that we would update the decorated"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bf659307_d573277f","line":7,"range":{"start_line":7,"start_character":1,"end_line":7,"end_character":9},"updated":"2018-04-12 08:12:33.000000000","message":"nit: This is not an optimization","commit_id":"425fe227deaa7eab64e9b7c1e1a4a96864d3dc67"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"3c86ea1c740965db59b0b58de690368589ac459d","unresolved":false,"context_lines":[{"line_number":12,"context_line":"\u0027expects_func_args\u0027 to ensure custom decorator\u0027s required"},{"line_number":13,"context_line":"parameters exist."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This comes from the identical method in Nova [1]"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[1]: https://github.com/openstack/nova/blob/15f1caf98a46ba0ab3f8365075c564e89f06eef3/nova/utils.py#L746"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"bf659307_b54d533c","line":15,"updated":"2018-04-12 08:12:33.000000000","message":"nit: I liked the original patch\u0027s commit message [1] more.\n\n[1] https://github.com/openstack/nova/commit/7cf7d4a6db69d9feefd8a3b120ec272528f90eea","commit_id":"425fe227deaa7eab64e9b7c1e1a4a96864d3dc67"}]}