🎨 Cleanup default Dangerfile

Signed-off-by: Peter Boling <[email protected]>

Author: pboling

Dangerfile

@@ -1,6 +1,6 @@
-# Sometimes it's a README fix, or something like that - which isn't relevant for
-# including in a project's CHANGELOG for example
-declared_trivial = github.pr_title.include? "#trivial"
+# Ideas...
+# 1. Check for hashtags in PR title, and disallow changes to changelog?
+#   e.g. github.pr_title.include? "#trivial"
 
 # Make it more obvious that a PR is a work in progress and shouldn't be merged yet
 warn("PR is classed as Work in Progress") if github.pr_title.include? "[WIP]"
@@ -9,5 +9,5 @@ warn("PR is classed as Work in Progress") if github.pr_title.include? "[WIP]"
 warn("Big PR") if git.lines_of_code > 500
 
 # Don't let testing shortcuts get into master by accident
-fail("fdescribe left in tests") if `grep -r fdescribe specs/ `.length > 1
-fail("fit left in tests") if `grep -r fit specs/ `.length > 1
+raise("fdescribe left in tests") if `grep -r fdescribe specs/ `.length > 1
+raise("fit left in tests") if `grep -r fit specs/ `.length > 1