Skip to content

Commit

Permalink
Merge pull request #184 from artsy/danger
Browse files Browse the repository at this point in the history
[Dev] Adds Danger
  • Loading branch information
orta committed Jun 23, 2017
2 parents a9b4174 + d16a776 commit 09fbec0
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
8 changes: 8 additions & 0 deletions danger/pr.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
const pr = danger.github.pr
const bodyAndTitle = (pr.body + pr.title).toLowerCase()
const trivialPR = bodyAndTitle.includes("trivial")

if (pr.assignee === null) {
const method = trivialPR ? warn : fail
method("Please assign someone to merge this PR, and optionally include people who should review.")
}
3 changes: 2 additions & 1 deletion tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"node_modules",
"dist",
"data",
"externals"
"externals",
"danger"
]
}

0 comments on commit 09fbec0

Please sign in to comment.