Use the original process.argv

.travis.yml

@@ -32,7 +32,8 @@ matrix:
     - yarn run link
     - echo "This is the real `danger ci` run on this repo"
     - DEBUG="*" danger ci --verbose
-    - DEBUG="*" danger pr https://github.com/danger/danger-js/pull/465 --verbose # To make sure it isn't broken again in a bad build
+    - echo "Validating that danger pr works as expected"
+    - DEBUG="*" danger pr https://github.com/danger/danger-js/pull/465 --verbose 
 
   # Create some fake projects at runtime
   - node_js: '7'

source/commands/danger.ts

@@ -32,10 +32,12 @@ program
     console.log("       http://danger.systems/js/reference.html")
   })
 
+// Commander mutates process.argv
+const originalProcessArgV = process.argv
 program.parse(process.argv)
 
 const showUpgradeNotice =
-  process.env.CI && ["init", "ci", "process", "pr", "--help"].some(cmd => process.argv.includes(cmd))
+  process.env.CI && ["init", "ci", "process", "pr", "--help"].some(cmd => originalProcessArgV.includes(cmd))
 
 if (showUpgradeNotice) {
   console.error("You may have updated from Danger 2.x -> 3.x without updating from `danger` to `danger ci`.")