Update patch set 3

Patch Set 3: Code-Review+2

(2 comments)

Patch-set: 3
Reviewer: Gerrit User 4146 <4146@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=+2, c6556c1e976b85175a109fc7583cf99336fa8290

e54d324370187039eb3cc9899d480ab70be58714

@@ -33,6 +33,40 @@
       "message": "I would say this is ok, because I don\u0027t think in this case I\u0027ve ever done anything but something ~ \"git rebase --abort; git fetch -a; git rebase -i origin/\u003cbranch, usually master\u003e\" and merged things directly like that.\n\nUnpacking those three commands for someone who a priori didn\u0027t know anything about git would probably take about a full day of tutorials and explanation about branches, rebases, staging commits, etc. :/ But I don\u0027t think that\u0027s really git-review\u0027s fault.\n\nIf you really don\u0027t know what\u0027s happening, resetting it so you can work your way through understanding the rebase process seems a good idea (at least you can follow tutorials, etc.).  If you do know what\u0027s happening, I imagine most like myself abort it and pull in changes manually anyway, because they want an up-to-date tree locally.  So ++ on the behaviour",
       "revId": "e54d324370187039eb3cc9899d480ab70be58714",
       "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "508381ea_4b8c2ca5",
+        "filename": "git_review/cmd.py",
+        "patchSetId": 3
+      },
+      "lineNbr": 990,
+      "author": {
+        "id": 4146
+      },
+      "writtenOn": "2024-03-05T15:36:49Z",
+      "side": 1,
+      "message": "Nit abort_rebase()\u0027s return values don\u0027t appear to be meaningful where used below. Could simplify this function to reflect that better by dropping these two explicit return values. But they also don\u0027t hurt anything.",
+      "revId": "e54d324370187039eb3cc9899d480ab70be58714",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
+    },
+    {
+      "unresolved": true,
+      "key": {
+        "uuid": "fbb7bb8f_3b881738",
+        "filename": "git_review/tests/test_git_review.py",
+        "patchSetId": 3
+      },
+      "lineNbr": 332,
+      "author": {
+        "id": 4146
+      },
+      "writtenOn": "2024-03-05T15:36:49Z",
+      "side": 1,
+      "message": "Would be good to have this test check the git rebase state after running git review. Also adding a new test for the -K flag to check that the rebase is not aborted. Basically a new test that is a small variant of this existing test.",
+      "revId": "e54d324370187039eb3cc9899d480ab70be58714",
+      "serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
     }
   ]
 }