86 lines
2.4 KiB
Plaintext
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
|
|
}
|
|
]
|
|
} |