Merge pull request #309 from bootstrap-ruby/danger

Set up Danger

Author: mattbrictson

.travis.yml

@@ -4,7 +4,7 @@ rvm:
   - 2.0
   - 2.1
   - 2.2
-  - 2.3.0
+  - 2.3.1
 gemfile:
   - gemfiles/4.0.gemfile
   - gemfiles/4.1.gemfile
@@ -18,6 +18,11 @@ matrix:
       gemfile: gemfiles/5.0.gemfile
     - rvm: 2.1
       gemfile: gemfiles/5.0.gemfile
+  include:
+    # Run Danger only once
+    - rvm: 2.3.1
+      gemfile: gemfiles/5.0.gemfile
+      script: bundle exec danger
 before_install:
   - gem install bundler --conservative --version '~> 1.10'
 before_script:

Dangerfile

@@ -0,0 +1,54 @@
+# Adapted from https://github.com/ruby-grape/danger/blob/master/Dangerfile
+# Q: What is a Dangerfile, anyway? A: See http://danger.systems/
+
+# ------------------------------------------------------------------------------
+# Additional pull request data
+# ------------------------------------------------------------------------------
+project_name = github.pr_json["base"]["repo"]["name"]
+pr_number = github.pr_json["number"]
+pr_url = github.pr_json["_links"]["html"]["href"]
+
+# ------------------------------------------------------------------------------
+# What changed?
+# ------------------------------------------------------------------------------
+has_lib_changes = !git.modified_files.grep(/^lib/).empty?
+has_test_changes = !git.modified_files.grep(/^test/).empty?
+has_changelog_changes = git.modified_files.include?("CHANGELOG.md")
+
+# ------------------------------------------------------------------------------
+# You've made changes to lib, but didn't write any tests?
+# ------------------------------------------------------------------------------
+if has_lib_changes && !has_test_changes
+  warn("There are code changes, but no corresponding tests. "\
+       "Please include tests if this PR introduces any modifications in "\
+       "#{project_name}'s behavior.",
+       :sticky => false)
+end
+
+# ------------------------------------------------------------------------------
+# Have you updated CHANGELOG.md?
+# ------------------------------------------------------------------------------
+if !has_changelog_changes && has_lib_changes
+  markdown <<-MARKDOWN
+Here's an example of a CHANGELOG.md entry (place it immediately under the `* Your contribution here!` line):
+
+```markdown
+* [##{pr_number}](#{pr_url}): #{github.pr_title} - [@#{github.pr_author}](https://github.com/#{github.pr_author}).
+```
+MARKDOWN
+  warn("Please update CHANGELOG.md with a description of your changes. "\
+       "If this PR is not a user-facing change (e.g. just refactoring), "\
+       "you can disregard this.", :sticky => false)
+end
+
+# ------------------------------------------------------------------------------
+# Did you remove the CHANGELOG's "Your contribution here!" line?
+# ------------------------------------------------------------------------------
+if has_changelog_changes
+  if IO.read("CHANGELOG.md").scan(/^\s*- Your contribution here/i).count < 2
+    fail(
+      "Please put the `- Your contribution here!` line back into CHANGELOG.md.",
+      :sticky => false
+    )
+  end
+end

gemfiles/5.0.gemfile

@@ -4,5 +4,6 @@ source "http://rubygems.org"
 
 gem "jquery-rails"
 gem "rails", "~> 5.0"
+gem "danger"
 
 gemspec :path => "../"