openinfra
/
interop
Code Issues Proposed changes

Update patch set 2 

Patch Set 2: Code-Review+1

(2 comments)

Patch-set: 2
Reviewer: Gerrit User 30025 <30025@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=+1
Attention: {"person_ident":"Gerrit User 30025 \u003c30025@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Vida Haririan replied on the change"}
Attention: {"person_ident":"Gerrit User 16643 \u003c16643@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Vida Haririan replied on the change"}
This commit is contained in:
Gerrit User 30025 2022-04-18 19:34:23 +00:00 committed by Gerrit Code Review
parent fac1207de7
commit aaa0b1b14d
1 changed files with 41 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
1cd6ffa317c8b0b85b5cb8e484831a5b187f3cb4
View File

@ -86,6 +86,23 @@
"revId": "1cd6ffa317c8b0b85b5cb8e484831a5b187f3cb4",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "96747e39_fd461ed0",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 2
},
"lineNbr": 0,
"author": {
"id": 30025
},
"writtenOn": "2022-04-18T19:34:23Z",
"side": 1,
"message": "Looks good, with one comment.",
"revId": "1cd6ffa317c8b0b85b5cb8e484831a5b187f3cb4",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
@ -108,6 +125,30 @@
},
"revId": "1cd6ffa317c8b0b85b5cb8e484831a5b187f3cb4",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "6938af04_83831e0a",
"filename": "tools/cross_check/exclude_file.txt",
"patchSetId": 2
},
"lineNbr": 31,
"author": {
"id": 30025
},
"writtenOn": "2022-04-18T19:34:23Z",
"side": 1,
"message": "Agreed!",
"parentUuid": "7c1e4683_c1d3ac2d",
"range": {
"startLine": 31,
"startChar": 36,
"endLine": 31,
"endChar": 42
},
"revId": "1cd6ffa317c8b0b85b5cb8e484831a5b187f3cb4",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}