Update patch set 1

Patch Set 1: Code-Review-1

(2 comments)

Thanks, but I think the URL for airship-berth is incorrect. Same comments apply to your other related open patches, so I won't review those until these concerns are clarified here. Else I think these changes are good to go!

Patch-set: 1
This commit is contained in:
Gerrit User 23186 2018-09-21 01:36:54 +00:00 committed by Gerrit Code Review
parent 9bf6a94633
commit 8276118120
1 changed files with 50 additions and 0 deletions

View File

@ -0,0 +1,50 @@
{
"comments": [
{
"key": {
"uuid": "3f79a3b5_8d2a52e4",
"filename": "/COMMIT_MSG",
"patchSetId": 1
},
"lineNbr": 9,
"author": {
"id": 23186
},
"writtenOn": "2018-09-21T01:36:54Z",
"side": 1,
"message": "LOCI?",
"range": {
"startLine": 9,
"startChar": 7,
"endLine": 9,
"endChar": 10
},
"revId": "ba432cffe6ad36893a414bf01135a6f0493d7950",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
},
{
"key": {
"uuid": "3f79a3b5_0df9627e",
"filename": "Dockerfile",
"patchSetId": 1
},
"lineNbr": 6,
"author": {
"id": 23186
},
"writtenOn": "2018-09-21T01:36:54Z",
"side": 1,
"message": "https://git.openstack.org/cgit/openstack/airship-berth\n\nNeeds cgit or it doesn\u0027t work",
"range": {
"startLine": 6,
"startChar": 39,
"endLine": 6,
"endChar": 88
},
"revId": "ba432cffe6ad36893a414bf01135a6f0493d7950",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}