138 lines
3.8 KiB
Plaintext
138 lines
3.8 KiB
Plaintext
{
|
|
"comments": [
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "220f57bb_3f9407c8",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 19,
|
|
"author": {
|
|
"id": 15435
|
|
},
|
|
"writtenOn": "2021-11-25T20:51:32Z",
|
|
"side": 1,
|
|
"message": "Usually Change-Id is be located with the story/task/bug/signed-off-by near the end of the commit message",
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "1ba7fa6b_9d1ace73",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 19,
|
|
"author": {
|
|
"id": 33278
|
|
},
|
|
"writtenOn": "2021-11-25T20:56:37Z",
|
|
"side": 1,
|
|
"message": "I think the \"----------\" might have confused the script and added the change-id there.",
|
|
"parentUuid": "220f57bb_3f9407c8",
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "a491045f_a3359182",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 19,
|
|
"author": {
|
|
"id": 33278
|
|
},
|
|
"writtenOn": "2021-11-25T20:59:21Z",
|
|
"side": 1,
|
|
"message": "Ack",
|
|
"parentUuid": "1ba7fa6b_9d1ace73",
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": true,
|
|
"key": {
|
|
"uuid": "4b130a35_7c7640d9",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 29,
|
|
"author": {
|
|
"id": 15435
|
|
},
|
|
"writtenOn": "2021-11-25T20:51:32Z",
|
|
"side": 1,
|
|
"message": "You could probably just include this one line as part of your: \u0027following message appears\u0027 comment above.\n\nHowever, since this is submitted for a task rather than a bug, I am fine with keeping this as well.",
|
|
"range": {
|
|
"startLine": 29,
|
|
"startChar": 0,
|
|
"endLine": 29,
|
|
"endChar": 73
|
|
},
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "01a63fa3_c0bc6875",
|
|
"filename": "/COMMIT_MSG",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 29,
|
|
"author": {
|
|
"id": 33278
|
|
},
|
|
"writtenOn": "2021-11-25T20:59:21Z",
|
|
"side": 1,
|
|
"message": "Ack",
|
|
"parentUuid": "4b130a35_7c7640d9",
|
|
"range": {
|
|
"startLine": 29,
|
|
"startChar": 0,
|
|
"endLine": 29,
|
|
"endChar": 73
|
|
},
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "971ea685_63a11bfa",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 15435
|
|
},
|
|
"writtenOn": "2021-11-25T20:51:32Z",
|
|
"side": 1,
|
|
"message": "changes seem fine to me. I have only minor nits related to the commit message",
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
},
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "ed1e9360_8f9f0ed4",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 3
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 34295
|
|
},
|
|
"writtenOn": "2021-11-26T19:03:37Z",
|
|
"side": 1,
|
|
"message": "up",
|
|
"revId": "2103d859de3f86a31f0a4f084a132bf34e937c1b",
|
|
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
|
}
|
|
]
|
|
} |