)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20870,"name":"Alex Kavanagh","email":"alex@ajkavanagh.co.uk","username":"ajkavanagh"},"change_message_id":"d9ebad2c0a511a12300c5c1446f86af60874c548","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c9f2dd96_1d728cdb","updated":"2023-02-10 10:49:05.000000000","message":"\u003e Patch Set 3:\n\u003e \n\u003e For that matter, your intent does not seem to be to revoke any refs/heads/* permissions for refs/heads/stable/* changes so there\u0027s no need for exclusiveGroupPermissions at all, nor for repeating anything already listed in the more general refs/heads/* access section.\n\u003e \n\u003e This overly complex pattern of marking labels exclusive and then readding some prior permissions dates back to when project core reviewers were not allowed to approve stable branch changes in most projects, and only the central stable branch maintainers team had that permission.\n\nAh, I see. Thanks for the clarification!  I\u0027ll update the review to remove those items.","commit_id":"cc3647c49b0652857b28d80318f12698386018ee"},{"author":{"_account_id":20870,"name":"Alex Kavanagh","email":"alex@ajkavanagh.co.uk","username":"ajkavanagh"},"change_message_id":"2c0f928ca739af10728beb115c5e42fe0bea1a1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1e449539_c20406b2","updated":"2023-02-10 11:54:54.000000000","message":"Actually, I think I can see what\u0027s going on; I\u0027ll try another patch.","commit_id":"f9b1d76233a32e94ae479c39f5ed67f79a89926d"},{"author":{"_account_id":20870,"name":"Alex Kavanagh","email":"alex@ajkavanagh.co.uk","username":"ajkavanagh"},"change_message_id":"b4c00df2149283facc28d564f337ed4c3904d293","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"045acd22_fdb2baed","updated":"2023-02-10 11:39:15.000000000","message":"I could do with some help with this one please.  It says it\u0027s \"not normalised\", which could mean \"wrong order\", or something else; are there some docs which explain what normalisation is required?  Thanks 😊","commit_id":"f9b1d76233a32e94ae479c39f5ed67f79a89926d"},{"author":{"_account_id":20870,"name":"Alex Kavanagh","email":"alex@ajkavanagh.co.uk","username":"ajkavanagh"},"change_message_id":"22681add221b01ec09a95062e88e7b8652d3424d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8f60e1d3_47f14757","updated":"2023-02-10 13:08:27.000000000","message":"\u003e Patch Set 6:\n\u003e \n\u003e If you\u0027re not using exclusiveGroupPermissions you don\u0027t need to repeat permissions in refs/heads/stable/* which already appear in refs/heads/* since they\u0027ll be inherited anyway. You can just list the additional permissions you want to grant in refs/heads/stable/* and it\u0027ll be the same end result with a much shorter ACL.\n\nAh, okay, I wasn\u0027t aware of that feature.  I\u0027ll update to make it smaller.  Thanks.","commit_id":"4d195b9135ae47204c22df1f0c6c0fb076bfd135"}]}