)]}'
{"playbooks/zuul/gerrit/increase-autogenerated-comment-width.html":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"61110df5e1338217a1fabacb3c29d268dc7394cd","unresolved":true,"context_lines":[{"line_number":1,"context_line":"\u003c!--"},{"line_number":2,"context_line":"Copyright (C) 2019 The Android Open Source Project"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":5,"context_line":"you may not use this file except in compliance with the License."}],"source_content_type":"text/html","patch_set":16,"id":"e00fabd0_c2e1107f","line":2,"updated":"2021-02-19 00:00:07.000000000","message":"It would be nice if we could write down the origin of the code. We\u0027re attributing it here, but if we need to look for updates or talk to maintainers for improvements I\u0027m not seeeing any location for that.","commit_id":"d7e33b5487c3ccc05db18cf6b2a35f7fe08475aa"},{"author":{"_account_id":31905,"name":"Guillaume Chauvel","email":"guillaume.chauvel@gmail.com","username":"gchauvel"},"change_message_id":"e2882efce43d3e4bc3232872aab20a5827376163","unresolved":true,"context_lines":[{"line_number":1,"context_line":"\u003c!--"},{"line_number":2,"context_line":"Copyright (C) 2019 The Android Open Source Project"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":5,"context_line":"you may not use this file except in compliance with the License."}],"source_content_type":"text/html","patch_set":16,"id":"3c0d199e_846bcf58","line":2,"in_reply_to":"e00fabd0_c2e1107f","updated":"2021-02-22 17:38:27.000000000","message":"This is a remainder from opendevtheme.html when I splitted the change to its own file.\nsee Patchset 4 -\u003e Patchset 5","commit_id":"d7e33b5487c3ccc05db18cf6b2a35f7fe08475aa"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"61110df5e1338217a1fabacb3c29d268dc7394cd","unresolved":true,"context_lines":[{"line_number":78,"context_line":"              if (comments[comment_idx].hasAttribute(\u0027data-message-id\u0027)) {"},{"line_number":79,"context_line":"                let comment_change_matched \u003d false;"},{"line_number":80,"context_line":"                while(change_idx \u003c this.change.messages.length) {"},{"line_number":81,"context_line":"                  if (this.change.messages[change_idx].id \u003d\u003d\u003d comments[comment_idx].getAttribute(\u0027data-message-id\u0027)) {"},{"line_number":82,"context_line":"                    if (this.change.messages[change_idx].tag \u0026\u0026 this.change.messages[change_idx].tag.startsWith(\u0027autogenerated\u0027) \u003d\u003d\u003d true) {"},{"line_number":83,"context_line":"                      let commentText \u003d comments[comment_idx].shadowRoot.querySelector(\u0027gr-formatted-text\u0027)"},{"line_number":84,"context_line":"                      commentText.style.setProperty(\"--gr-formatted-text-prose-max-width\", \u0027100%\u0027);"}],"source_content_type":"text/html","patch_set":16,"id":"572d8955_7ae79983","line":81,"updated":"2021-02-19 00:00:07.000000000","message":"It would be nice to have a comment here explaining why we have to cross check the comments list and messages list here. This nested loop appears to be in service of this check, but it isn\u0027t clear why we couldn\u0027t just go over all messages, check they are autogenerated and modify their properties. That would be much simpler.","commit_id":"d7e33b5487c3ccc05db18cf6b2a35f7fe08475aa"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"240e36b7deeeac430d5be2e130496e802ecd704c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"// Copyright (c) 2021 OpenStack Foundation"},{"line_number":2,"context_line":"//"},{"line_number":3,"context_line":"// Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":4,"context_line":"// you may not use this file except in compliance with the License."}],"source_content_type":"text/html","patch_set":19,"id":"7a2f6a0e_dbb8b46b","line":1,"updated":"2021-02-22 17:31:29.000000000","message":"Note, if this plugin code was written by you then the copyright would be for you or in many cases your employer. The OSF applies copyright to things its employees have done.","commit_id":"e780a81a3ee2ff23ae7a0cf16d73371f22341876"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"240e36b7deeeac430d5be2e130496e802ecd704c","unresolved":true,"context_lines":[{"line_number":81,"context_line":"            iterate over the DOM elements first, and look for a match in the backend data"},{"line_number":82,"context_line":"            using \u0027data-message-id\u0027"},{"line_number":83,"context_line":"            Not all DOM elements have a \u0027data-message-id\u0027 attribute, we must filter then."},{"line_number":84,"context_line":"            */"},{"line_number":85,"context_line":"            for (let comment_idx \u003d 0;"},{"line_number":86,"context_line":"                 comments !\u003d\u003d null \u0026\u0026 comment_idx \u003c comments.length;"},{"line_number":87,"context_line":"                 ++comment_idx) {"}],"source_content_type":"text/html","patch_set":19,"id":"52a2a617_f68ad3e8","line":84,"updated":"2021-02-22 17:31:29.000000000","message":"This helps significantly. Thank you.","commit_id":"e780a81a3ee2ff23ae7a0cf16d73371f22341876"}]}