ui fixes

Author: ds300

integration-tests/happy-path-npm/__snapshots__/happy-path-npm.test.ts.snap

@@ -3,10 +3,9 @@
 exports[`Test happy-path-npm: making patch 1`] = `
 "SNAPSHOT: making patch
 patch-package 0.0.0
-✔ Creating temporary folder
-✔ Making tmp package.json
-✔ Installing left-pad@1.1.3 with npm
-✔ Diffing your files with clean files
+• Creating temporary folder
+• Installing left-pad@1.1.3 with npm
+• Diffing your files with clean files
 ✔ Created file patches/left-pad+1.1.3.patch
 END SNAPSHOT"
 `;

integration-tests/happy-path-yarn/__snapshots__/happy-path-yarn.test.ts.snap

@@ -3,10 +3,9 @@
 exports[`Test happy-path-yarn: making patch 1`] = `
 "SNAPSHOT: making patch
 patch-package 0.0.0
-✔ Creating temporary folder
-✔ Making tmp package.json
-✔ Installing left-pad@1.1.3 with yarn
-✔ Diffing your files with clean files
+• Creating temporary folder
+• Installing left-pad@1.1.3 with yarn
+• Diffing your files with clean files
 ✔ Created file patches/left-pad+1.1.3.patch
 END SNAPSHOT"
 `;

integration-tests/nested-packages/__snapshots__/nested-packages.test.ts.snap

@@ -3,10 +3,9 @@
 exports[`Test nested-packages: create the patch 1`] = `
 "SNAPSHOT: create the patch
 patch-package 0.0.0
-✔ Creating temporary folder
-✔ Making tmp package.json
-✔ Installing string-width@2.1.1 with yarn
-✔ Diffing your files with clean files
+• Creating temporary folder
+• Installing string-width@2.1.1 with yarn
+• Diffing your files with clean files
 ✔ Created file patches/wrap-ansi++string-width+2.1.1.patch
 END SNAPSHOT"
 `;

integration-tests/nested-scoped-packages/__snapshots__/nested-scoped-packages.test.ts.snap

@@ -3,10 +3,9 @@
 exports[`Test nested-scoped-packages: create the patch 1`] = `
 "SNAPSHOT: create the patch
 patch-package 0.0.0
-✔ Creating temporary folder
-✔ Making tmp package.json
-✔ Installing @types/angular@1.6.53 with yarn
-✔ Diffing your files with clean files
+• Creating temporary folder
+• Installing @types/angular@1.6.53 with yarn
+• Diffing your files with clean files
 ✔ Created file patches/@microsoft+mezzurite-core++@types+angular+1.6.53.patch
 END SNAPSHOT"
 `;

src/index.ts

@@ -11,7 +11,13 @@ import { join } from "./path"
 
 const appPath = getAppRootPath()
 const argv = minimist(process.argv.slice(2), {
-  boolean: ["use-yarn", "case-sensitive-path-filtering", "reverse"],
+  boolean: [
+    "use-yarn",
+    "case-sensitive-path-filtering",
+    "reverse",
+    "help",
+    "version",
+  ],
   string: ["patch-dir"],
 })
 const packageNames = argv._
@@ -22,7 +28,9 @@ console.log(
   require(join(__dirname, "../package.json")).version,
 )
 
-if (argv.help || argv.h) {
+if (argv.version || argv.v) {
+  // noop
+} else if (argv.help || argv.h) {
   printHelp()
 } else {
   if (packageNames.length) {

src/makePatch.ts

@@ -1,4 +1,4 @@
-import { green } from "chalk"
+import { green, grey } from "chalk"
 import { join, dirname, resolve } from "./path"
 import { spawnSafeSync } from "./spawnSafe"
 import { PackageManager } from "./detectPackageManager"
@@ -89,10 +89,9 @@ export const makePatch = (
   try {
     const patchesDir = join(appPath, patchDir)
 
-    console.info(green("✔"), "Creating temporary folder")
+    console.info(grey("•"), "Creating temporary folder")
 
     // make a blank package.json
-    console.info(green("✔"), "Making tmp package.json")
     mkdirpSync(tmpRepoNpmRoot)
     writeFileSync(
       tmpRepoPackageJsonPath,
@@ -105,15 +104,15 @@ export const makePatch = (
 
     if (packageManager === "yarn") {
       console.info(
-        green("✔"),
+        grey("•"),
         `Installing ${packageDetails.name}@${packageVersion} with yarn`,
       )
       spawnSafeSync(`yarn`, ["install", "--ignore-engines"], {
         cwd: tmpRepoNpmRoot,
       })
     } else {
       console.info(
-        green("✔"),
+        grey("•"),
         `Installing ${packageDetails.name}@${packageVersion} with npm`,
       )
       spawnSafeSync(`npm`, ["i"], { cwd: tmpRepoNpmRoot })
@@ -131,7 +130,7 @@ export const makePatch = (
     rimraf(join(tmpRepoPackagePath, "node_modules"))
 
     // commit the package
-    console.info(green("✔"), "Diffing your files with clean files")
+    console.info(grey("•"), "Diffing your files with clean files")
     writeFileSync(join(tmpRepo.name, ".gitignore"), "!/node_modules\n\n")
     git("init")
     git("config", "--local", "user.name", "patch-package")