)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"aa03901940a572fc1cb12682704002f3af6234cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3867c785_55264748","updated":"2024-01-05 14:31:57.000000000","message":"Looks reasonable to me","commit_id":"d14cde240297d6253c32b0310d4e0ca00f231e97"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6727d262adb444d9a5d7da421a4b2b908d450668","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a3b1cb6a_19427436","updated":"2024-02-29 16:32:34.000000000","message":"Not approving because I hope fungi can review this. But feel free to approve if he doesn\u0027t get to it in the near future.","commit_id":"80fa9e7a2c43a448cf2efae3107609d2425059a6"}],"roles/copy-release-tools-scripts/files/release-tools/make_branch.sh":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"28e866755b30fab64f587b58dc893fdc6e68644b","unresolved":false,"context_lines":[{"line_number":57,"context_line":""},{"line_number":58,"context_line":"# Skip stable/\u003cseries\u003e or unmaintained/\u003cseries\u003e branch creation in case"},{"line_number":59,"context_line":"# \u003cseries\u003e-eol tag exists"},{"line_number":60,"context_line":"if $(git tag | grep ${NEW_BRANCH//@(stable\\/|unmaintained\\/)}-eol \u003e/dev/null); then"},{"line_number":61,"context_line":"    echo \"A ${NEW_BRANCH//@(stable\\/|unmaintained\\/)}-eol tag already exists !\""},{"line_number":62,"context_line":"    cleanup_and_exit"},{"line_number":63,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"b5c48fa7_8c1e2ab6","line":60,"updated":"2024-02-29 16:47:04.000000000","message":"Nit: Keep in mind that grep has a -q option to not emit matches on stdout, which would save you this redirect. It still exits 0 or 1 depending on whether a match was found, so you can use it exactly as you want here.","commit_id":"80fa9e7a2c43a448cf2efae3107609d2425059a6"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"28e866755b30fab64f587b58dc893fdc6e68644b","unresolved":false,"context_lines":[{"line_number":63,"context_line":"fi"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"# Skip stable/\u003cseries\u003e branch creation in case \u003cseries\u003e-eom tag exists"},{"line_number":66,"context_line":"if $(git tag | grep ${NEW_BRANCH#stable/}-eom \u003e/dev/null); then"},{"line_number":67,"context_line":"    echo \"A ${NEW_BRANCH#stable/}-eom tag already exists !\""},{"line_number":68,"context_line":"    cleanup_and_exit"},{"line_number":69,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"02cc9de0_dfa7cad6","line":66,"updated":"2024-02-29 16:47:04.000000000","message":"Nit: For same possible use of -q.","commit_id":"80fa9e7a2c43a448cf2efae3107609d2425059a6"}]}