56 lines
1.7 KiB
Plaintext
56 lines
1.7 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "306dd0f2_a0f5b77b",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 11
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 13252
|
|
},
|
|
"writtenOn": "2024-03-12T19:54:33Z",
|
|
"side": 1,
|
|
"message": "question about the manpage wording, but wouldn\u0027t block this",
|
|
"revId": "43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "cc8c6284_b1752245",
|
|
"filename": "git-review.1",
|
|
"patchSetId": 11
|
|
},
|
|
"lineNbr": 166,
|
|
"author": {
|
|
"id": 13252
|
|
},
|
|
"writtenOn": "2024-03-12T19:54:33Z",
|
|
"side": 1,
|
|
"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?",
|
|
"revId": "43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "2f5270ca_41a471e2",
|
|
"filename": "git-review.1",
|
|
"patchSetId": 11
|
|
},
|
|
"lineNbr": 166,
|
|
"author": {
|
|
"id": 5263
|
|
},
|
|
"writtenOn": "2024-03-12T20:01:06Z",
|
|
"side": 1,
|
|
"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.",
|
|
"parentUuid": "cc8c6284_b1752245",
|
|
"revId": "43c92f2e7cc8ecad14b2b704adfc3db5cf8cd848",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |