Update patch set 2
Patch Set 2: Code-Review+1 (2 comments) Minor comments. Giving +1 in case people disagree with them since I'm fine going ahead with no change. Patch-set: 2 Label: Code-Review=+1
This commit is contained in:
parent
e6b0764eb4
commit
6b5b6b7fd3
|
@ -0,0 +1,38 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "da0c15f0_016d4998",
|
||||
"filename": "ocf/NovaCompute",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 125,
|
||||
"author": {
|
||||
"id": 4460
|
||||
},
|
||||
"writtenOn": "2016-03-25T13:38:12Z",
|
||||
"side": 1,
|
||||
"message": "With the explanation, I think the \"Use at your own risk!\" bit is not needed anymore, so I\u0027d drop that.",
|
||||
"revId": "c099d2f5590194598f1a29d64e846e2f64464555",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "da0c15f0_c1f991ca",
|
||||
"filename": "ocf/NovaEvacuate",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 125,
|
||||
"author": {
|
||||
"id": 4460
|
||||
},
|
||||
"writtenOn": "2016-03-25T13:38:12Z",
|
||||
"side": 1,
|
||||
"message": "Same here.",
|
||||
"revId": "c099d2f5590194598f1a29d64e846e2f64464555",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue