Update patch set 5

Patch Set 5:

(1 comment)

Patch-set: 5
This commit is contained in:
Gerrit User 33549 2024-02-29 21:43:13 +00:00 committed by Gerrit Code Review
parent 87a6e265ce
commit 951c2c10d9
1 changed files with 27 additions and 0 deletions

View File

@ -0,0 +1,27 @@
{
"comments": [
{
"unresolved": true,
"key": {
"uuid": "7d15489b_20655534",
"filename": "distributedcloud-client/dcmanagerclient/api/v1/phased_subcloud_deploy_manager.py",
"patchSetId": 5
},
"lineNbr": 32,
"author": {
"id": 33549
},
"writtenOn": "2024-02-29T21:43:13Z",
"side": 1,
"message": "I think these would all be cleaner and simpler without the intermediate variable:\n\n with open(v, \"rb\") as file:\n fields.update({k: (v, file.read())})\n \nSame comment for the bulk of the changes in this commit.",
"range": {
"startLine": 31,
"startChar": 0,
"endLine": 32,
"endChar": 49
},
"revId": "92597a3d2372ff9c3f075c422dfa92f6ad51a732",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}