diff --git a/5bb72719fce93a93cf79169436088012281280d7 b/5bb72719fce93a93cf79169436088012281280d7 new file mode 100644 index 0000000..d5ed419 --- /dev/null +++ b/5bb72719fce93a93cf79169436088012281280d7 @@ -0,0 +1,67 @@ +{ + "comments": [ + { + "key": { + "uuid": "bfb3d3c7_6903d074", + "filename": "fixtures_git/gitfixture.py", + "patchSetId": 1 + }, + "lineNbr": 295, + "author": { + "id": 5557 + }, + "writtenOn": "2019-05-22T13:51:52Z", + "side": 1, + "message": "I think a docstring would be useful here even just to note that the username/password are not used for the initial clone, even if set.", + "revId": "5bb72719fce93a93cf79169436088012281280d7", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543", + "unresolved": false + }, + { + "key": { + "uuid": "bfb3d3c7_8995e4e6", + "filename": "fixtures_git/gitfixture.py", + "patchSetId": 1 + }, + "lineNbr": 315, + "author": { + "id": 5557 + }, + "writtenOn": "2019-05-22T13:51:52Z", + "side": 1, + "message": "Change to:\n\n if self.username:\n\n?\n\n(Setting self.username to \u0027\u0027 could result in a badly-formatted auth component of the recomposed url)", + "range": { + "startLine": 315, + "startChar": 8, + "endLine": 315, + "endChar": 36 + }, + "revId": "5bb72719fce93a93cf79169436088012281280d7", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543", + "unresolved": false + }, + { + "key": { + "uuid": "bfb3d3c7_c96d1c99", + "filename": "fixtures_git/gitfixture.py", + "patchSetId": 1 + }, + "lineNbr": 317, + "author": { + "id": 5557 + }, + "writtenOn": "2019-05-22T13:51:52Z", + "side": 1, + "message": "This seems redundant since auth, and presumably url_parts[1], will always be set.", + "range": { + "startLine": 317, + "startChar": 30, + "endLine": 317, + "endChar": 37 + }, + "revId": "5bb72719fce93a93cf79169436088012281280d7", + "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543", + "unresolved": false + } + ] +} \ No newline at end of file