Merged by Peril

Lets --json or --js not require a dangerfile
danger · Feb 3, 2019 · 854655d · 854655d
2 parents 9c9d391 + c1e524a
commit 854655d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,8 @@
 
 <!-- Your comment below this -->
 
+- `danger pr` with `--json` or --js` now don't require a Dangerfile to be present - [@orta][]
+
 # 7.0.5
 
 - Remove dead links to example danger files. - [@stevemoser][]

diff --git a/source/commands/danger-pr.ts b/source/commands/danger-pr.ts
@@ -79,7 +79,7 @@ if (program.args.length === 0) {
 
       console.log(`Starting Danger PR on ${pr.repo}#${pr.pullRequestNumber}`)
 
-      if (customProcess || validateDangerfileExists(dangerfilePath(program))) {
+      if (customProcess || app.js || app.json || validateDangerfileExists(dangerfilePath(program))) {
         if (!customProcess) {
           d(`executing dangerfile at ${dangerfilePath(program)}`)
         }