d8c8697b91
Patch Set 9:
(2 comments)
Patch-set: 9
Attention: {"person_ident":"Gerrit User 4127492 \u003c4127492@173816e5-2b9a-37c3-8a2e-48639d4f1153\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_4127492\u003e replied on the change"}
Attention: {"person_ident":"Gerrit User 1002666 \u003c1002666@173816e5-2b9a-37c3-8a2e-48639d4f1153\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_4127492\u003e replied on the change"}
21 lines
No EOL
877 B
Text
21 lines
No EOL
877 B
Text
{
|
|
"comments": [
|
|
{
|
|
"unresolved": false,
|
|
"key": {
|
|
"uuid": "aac8f3d7_9fc81e31",
|
|
"filename": "/PATCHSET_LEVEL",
|
|
"patchSetId": 9
|
|
},
|
|
"lineNbr": 0,
|
|
"author": {
|
|
"id": 4127492
|
|
},
|
|
"writtenOn": "2025-06-04T03:15:57Z",
|
|
"side": 1,
|
|
"message": "In patchset 9, I updated the code to use the actual `opt` passed to `Execute()` instead of constructing a dummy. I also ensured the required fields (`bypass_hooks`, `allow_all_hooks`, and `ignore_hooks`) are set with safe defaults if not present.\n\nThis keeps the implementation in line with `RepoHook.FromSubcmd()` expectations without changing the CLI interface.\n\nThanks again for the feedback — it\u0027s cleaner now!",
|
|
"revId": "50620d2a2d6324ca0e8e34329d13626f064ceee4",
|
|
"serverId": "173816e5-2b9a-37c3-8a2e-48639d4f1153"
|
|
}
|
|
]
|
|
} |