)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35021,"name":"Mark Fedorov","email":"marcfedorow@gmail.com","username":"marcfedorow"},"change_message_id":"81d9cf59f9d8b5e080aee1efea534006979f00d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"30f6b202_b25e68c1","updated":"2022-07-11 13:48:18.000000000","message":"\u003e Patch Set 3:\n\u003e \n\u003e Do also please remember to add a test so this feature doesn\u0027t regress. You should hopefully be able to base it on test_git_review_re() with only some minor adjustments.\n\nDone. Thank you","commit_id":"527bf30e12becfa5119b55c48a1d3a29292d5203"},{"author":{"_account_id":35021,"name":"Mark Fedorov","email":"marcfedorow@gmail.com","username":"marcfedorow"},"change_message_id":"8367e4dc9eed3391b8be7cc5a4de03cd56b25498","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0e216804_4c20b6c5","updated":"2023-03-09 15:33:23.000000000","message":"I didn\u0027t get how to add a release note.\nHere -- releasenotes/notes -- filenames contain some hash which I have no idea how to obtain.","commit_id":"6f1f0f0ce7fd2f20a87512b02731cb899c6d486f"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f43bcedda2ffe218f4bb3a76d1be7200cf40e732","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"bbee60b0_6701df66","updated":"2024-01-30 08:03:11.000000000","message":"oops, I only wanted to test this patch, I expected gerrit to reject my no-op change, but didn\u0027t remember the rebase I did to get tox working\n\nregarding the release note, have a look at https://docs.openstack.org/reno/latest/user/usage.html. the hash is just to avoid collisions afaict, nothing fancy","commit_id":"2f67aacb690f855f77dcd1da3e447d3b955df1b8"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6f9fd506993f0d0ca76e0f18bc79c082440923b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"306dd0f2_a0f5b77b","updated":"2024-03-12 19:54:33.000000000","message":"question about the manpage wording, but wouldn\u0027t block this","commit_id":"43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848"}],"git-review.1":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6f9fd506993f0d0ca76e0f18bc79c082440923b0","unresolved":true,"context_lines":[{"line_number":163,"context_line":".It Fl W , Fl \\-ready"},{"line_number":164,"context_line":"Send patch that is already work in progress as ready for review. Gerrit versions \u003e\u003d 2.15"},{"line_number":165,"context_line":".It Fl \\-cc Ar reviewer ..."},{"line_number":166,"context_line":"Carbon copy one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":167,"context_line":".It Fl \\-reviewers Ar reviewer ..."},{"line_number":168,"context_line":"Subscribe one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":169,"context_line":".It Fl \\-notify Ar type"}],"source_content_type":"text/troff","patch_set":11,"id":"cc8c6284_b1752245","line":166,"updated":"2024-03-12 19:54:33.000000000","message":"Is there a better term than \"reviewer\" in this case? because CC: in this case is explicitly different from \"reviewers\". like just \"users\" as in the reno?","commit_id":"43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a887325a108697d4fe1e227066f147be48d657cf","unresolved":true,"context_lines":[{"line_number":163,"context_line":".It Fl W , Fl \\-ready"},{"line_number":164,"context_line":"Send patch that is already work in progress as ready for review. Gerrit versions \u003e\u003d 2.15"},{"line_number":165,"context_line":".It Fl \\-cc Ar reviewer ..."},{"line_number":166,"context_line":"Carbon copy one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":167,"context_line":".It Fl \\-reviewers Ar reviewer ..."},{"line_number":168,"context_line":"Subscribe one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":169,"context_line":".It Fl \\-notify Ar type"}],"source_content_type":"text/troff","patch_set":11,"id":"47a71fee_c2a207ef","line":166,"in_reply_to":"2f5270ca_41a471e2","updated":"2024-03-13 12:14:36.000000000","message":"To me, this is similar to how \"To:\" vs. \"Cc:\" are (or can be) used in mails, \"please handle (review) this\" vs. \"please have a look, this might be interesting for you, but no immediate action required\". Note that submitting a comment via the UI that doesn\u0027t set any vote will also add the submitter to the CC list instead of the reviewers list.\n\nBut maybe I\u0027m also overthinking this, since Gerrit is a code review system, any user will also be a reviewer by definition. So let\u0027s just proceed, or do you want to keep this for the next release?","commit_id":"43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"67f0ef0c642ed3b04f0229089641b2c8fad97dc8","unresolved":false,"context_lines":[{"line_number":163,"context_line":".It Fl W , Fl \\-ready"},{"line_number":164,"context_line":"Send patch that is already work in progress as ready for review. Gerrit versions \u003e\u003d 2.15"},{"line_number":165,"context_line":".It Fl \\-cc Ar reviewer ..."},{"line_number":166,"context_line":"Carbon copy one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":167,"context_line":".It Fl \\-reviewers Ar reviewer ..."},{"line_number":168,"context_line":"Subscribe one or more reviewers to the uploaded patch sets.  Reviewers should be identifiable by Gerrit (usually use their Gerrit username or email address)."},{"line_number":169,"context_line":".It Fl \\-notify Ar type"}],"source_content_type":"text/troff","patch_set":11,"id":"2f5270ca_41a471e2","line":166,"in_reply_to":"cc8c6284_b1752245","updated":"2024-03-12 20:01:06.000000000","message":"It\u0027s unclear to me what the purpose of Cc\u0027ing a user is if not to review the change, but I can try to find different wording to help differentiate it.","commit_id":"43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848"}]}
