feat(pop): allow unescaped negative patch offsets

This allows, for example, `stg pop -1` to pop the patch preceding the
topmost patch. Previously, a negative relative patch offset would have
had to been escaped, e.g. `stg pop '\-1'`

Author: jpgrayson

src/cmd/pop.rs

@@ -50,6 +50,7 @@ fn make() -> clap::Command {
                 .help("Patches to pop")
                 .value_name("patch")
                 .num_args(1..)
+                .allow_negative_numbers(true)
                 .value_parser(clap::value_parser!(PatchRange))
                 .conflicts_with_all(["all", "number"]),
         )

t/t1208-push-and-pop.sh

@@ -42,7 +42,7 @@ test_expect_success 'Check prev, next, and top with all applied' '
 '
 
 test_expect_success 'Pop three patches' '
-    stg pop -n 3 &&
+    stg pop -n3 &&
     [ "$(echo $(stg series --applied --noprefix))" = "p0 p1 p2 p3 p4 p5 p6" ] &&
     [ "$(echo $(stg series --unapplied --noprefix))" = "p7 p8 p9" ]
 '
@@ -126,6 +126,12 @@ test_expect_success 'Push two patches in reverse' '
     [ "$(git notes show)" = "note7" ]
 '
 
+test_expect_success 'Pop relative patch offset' '
+    stg pop -4 &&
+    [ "$(echo $(stg series --applied --noprefix))" = "p0 p1 p2 p3 p5 p6 p8 p7" ] &&
+    [ "$(echo $(stg series --unapplied --noprefix))" = "p4 p9" ]
+'
+
 test_expect_success 'Attempt to push already applied patches' '
     command_error stg push p0..p2 2>err &&
     grep -e "patch \`p0\` is already applied" err &&