1.8.23: check branch earlier

disruptek · Sep 12, 2020 · b9f0c55 · b9f0c55
1 parent c7ffe40
commit b9f0c55
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/bump.nim b/bump.nim
@@ -459,6 +459,16 @@ proc bump*(minor = false; major = false; patch = true; release = false;
     debug sought.message
     target = sought.found.get
 
+  # if we're not on the master branch, let's just bail for now
+  let
+    branch = appearsToBeMasterBranch()
+  if branch.isNone:
+    crash "uh oh; i cannot tell if i'm on the master branch"
+  elif not branch.get:
+    crash "i'm afraid to modify any branch that isn't master"
+  else:
+    debug "good; this appears to be the master branch"
+
   # make a temp file in an appropriate spot, with a significant name
   let
     temp = createTemporaryFile(target.package, dotNimble)
@@ -533,16 +543,6 @@ proc bump*(minor = false; major = false; patch = true; release = false;
     debug "dry run and done"
     return
 
-  # if we're not on the master branch, let's just bail for now
-  let
-    branch = appearsToBeMasterBranch()
-  if branch.isNone:
-    crash "uh oh; i cannot tell if i'm on the master branch"
-  elif not branch.get:
-    crash "i'm afraid to modify any branch that isn't master"
-  else:
-    debug "good; this appears to be the master branch"
-
   # copy the new .nimble over the old one
   try:
     debug &"copying {temp} over {target}"

diff --git a/bump.nimble b/bump.nimble
@@ -1,4 +1,4 @@
-version = "1.8.22"
+version = "1.8.23"
 author = "disruptek"
 description = "a tiny tool to bump nimble versions"
 license = "MIT"