Merge pull request #916 from danger/fix_ci

Blocks PRs which change the package version, and fixes CI

.travis.yml

@@ -67,7 +67,7 @@ matrix:
           source/platforms/git/_tests/local_dangerfile_example.ts || echo "Skipping Danger Local for non PR run"'
 
     # Create some fake projects at runtime
-    - node_js: "8"
+    - node_js: "10.16"
       script:
         - echo "This is only for Integration tests on two blank projects"
         - yarn build

CHANGELOG.md

@@ -15,6 +15,8 @@
 
 <!-- Your comment below this -->
 
+<!-- Your comment above this -->
+
 - Improve docs for GitHub Actions - [@nguyenhuy]
 
 # 9.1.8

dangerfile.ts

@@ -38,6 +38,12 @@ export default async () => {
   // Some libraries
   await yarn()
   await jest()
+
+  // Don't have folks setting the package json version
+  const packageDiff = await danger.git.JSONDiffForFile("package.json")
+  if (packageDiff.version && danger.github.pr.user.login !== "orta") {
+    fail("Please don't make package version changes")
+  }
 }
 
 // Re-run the git push hooks