fix: add simple-git dependency, use it when checking for tracked files (

#99)

Progress toward #86
Progress toward #98

First step in moving off of nodegit, mostly taken from @refack's branch from #86.

package.json

@@ -61,6 +61,7 @@
     "mz": "^2.4.0",
     "nodegit": "^0.16.0",
     "opn": "^4.0.2",
-    "require-uncached": "^1.0.2"
+    "require-uncached": "^1.0.2",
+    "simple-git": "^1.70.0"
   }
 }

src/convert.js

@@ -8,7 +8,7 @@ import runWithProgressBar from './runner/runWithProgressBar';
 import CLIError from './util/CLIError';
 import execLive from './util/execLive';
 import getFilesUnderPath from './util/getFilesUnderPath';
-import gitTrackedStatus from './util/gitTrackedStatus';
+import isWorktreeEmpty from './util/isWorktreeEmpty';
 import makeCommit from './util/makeCommit';
 import pluralize from './util/pluralize';
 
@@ -164,8 +164,7 @@ Re-run with the "check" command for more details.`);
 }
 
 async function assertGitWorktreeClean() {
-  let changedFiles = await gitTrackedStatus();
-  if (changedFiles.length) {
+  if (!await isWorktreeEmpty()) {
     throw new CLIError(`\
 You have modifications to your git worktree.
 Please revert or commit them before running convert.`);

src/util/isWorktreeEmpty.js

@@ -0,0 +1,9 @@
+import git from 'simple-git/promise';
+
+/**
+ * Determine if there are any uncommitted changes in the git state.
+ */
+export default async function isWorktreeEmpty() {
+  const status = await git().status();
+  return status.files.length === 0;
+}