validations-libs/e90cfdaef669dde6092d848f693...

86 lines
2.4 KiB
Plaintext

{
"comments": [
{
"key": {
"uuid": "6539d51c_a87ff3d0",
"filename": "validations_libs/cli/history.py",
"patchSetId": 7
},
"lineNbr": 40,
"author": {
"id": 28223
},
"writtenOn": "2021-05-04T10:14:24Z",
"side": 1,
"message": "Wondering if we shouldn\u0027t enforce a default in order to avoid endless listing? what about 15?",
"revId": "e90cfdaef669dde6092d848f6931b756f226032d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": true
},
{
"key": {
"uuid": "f231dad1_5cf4696b",
"filename": "validations_libs/cli/history.py",
"patchSetId": 7
},
"lineNbr": 40,
"author": {
"id": 32926
},
"writtenOn": "2021-05-04T11:29:47Z",
"side": 1,
"message": "The idea was to keep default behavior the same as it was until now.\n\nBut limiting the output to some number by default seems to be sensible.\nI\u0027ll put it for consideration",
"parentUuid": "6539d51c_a87ff3d0",
"revId": "e90cfdaef669dde6092d848f6931b756f226032d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": true
},
{
"key": {
"uuid": "a42037fb_2f65ba9d",
"filename": "validations_libs/cli/history.py",
"patchSetId": 7
},
"lineNbr": 53,
"author": {
"id": 28223
},
"writtenOn": "2021-05-04T10:14:24Z",
"side": 1,
"message": "you might do the same check as shown in the doc string (\u003e 0). For the sake of consistency.",
"range": {
"startLine": 53,
"startChar": 67,
"endLine": 53,
"endChar": 70
},
"revId": "e90cfdaef669dde6092d848f6931b756f226032d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": true
},
{
"key": {
"uuid": "4e6052d6_930876df",
"filename": "validations_libs/cli/history.py",
"patchSetId": 7
},
"lineNbr": 53,
"author": {
"id": 32926
},
"writtenOn": "2021-05-04T11:29:47Z",
"side": 1,
"message": "Ack",
"parentUuid": "a42037fb_2f65ba9d",
"range": {
"startLine": 53,
"startChar": 67,
"endLine": 53,
"endChar": 70
},
"revId": "e90cfdaef669dde6092d848f6931b756f226032d",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
"unresolved": false
}
]
}