danger 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e9d3bed0d3e1f30e4a29d4eef41445ccd9f5b00d
-  data.tar.gz: eaf83e5b95373942ac5f56ecaa1cd7bfdb25cd5a
+  metadata.gz: c2df0507da13cc40eb7f1d93256d1c5da4363ee0
+  data.tar.gz: 89be5c58b12e017286d8d0dedafcd5adc80f05bf
 SHA512:
-  metadata.gz: 340a33da5826421c3e7ddcfe7b8d7f8f23ac12260da1d81933e672df55987a1b341d23f9fcfe373c1177e541a8b45e194c74df7ffeeae7fc382dd6ee8baf12ea
-  data.tar.gz: 8e5f6adafea5d67c8b1799123ff02cfb0752dbfc4d1113efec734c2c79dace8bab00f8c68848e2ab441f44b8a7b46125bbac15277b7acf1a8e851532c506ce8b
+  metadata.gz: 44a07b2b9aca7eac7d487576fc1efb3332e3e978120c49046428ae623e8b76c5ed965ec44b5482bb3b5024206301490933118537275723ba1acdc8d14fdc6690
+  data.tar.gz: 10c85b4daeb7feff0c22d8c3b39746aaa14963bba02788b9a83d310334670c857eb4a5d651801e74803b68c4d9c7a92b373fcaa8747d5844a4fcd7653ef259aa

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2015 Orta
+Copyright (c) 2015 Orta, Felix Krause
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,8 +1,23 @@
 # Danger :no_entry_sign:
 
+[![License](http://img.shields.io/badge/license-MIT-green.svg?style=flat)](https://github.com/orta/danger/blob/master/LICENSE)
+[![Gem](https://img.shields.io/gem/v/danger.svg?style=flat)](http://rubygems.org/gems/danger)
+
 Formalize your Pull Request etiquette.
 
-*Note:* Not ready for public usage yet. Still needs work to ensure Travis/CircleCI hook up correctly.
+*Note:* Not ready for public usage yet. Work in progress
+
+-------
+<p align="center">
+    <a href="#installation">Installation</a> &bull; 
+    <a href="#usage">Usage</a> &bull; 
+    <a href="#dsl">DSL</a> &bull; 
+    <a href="#constraints">Constraints</a> &bull; 
+    <a href="#advanced">Advanced</a> &bull; 
+    <a href="#contributing">Contributing</a>
+</p>
+
+-------
 
 ## Installation
 
@@ -12,31 +27,27 @@ Add this line to your application's [Gemfile](https://guides.cocoapods.org/using
 gem 'danger'
 ```
 
+and then run the following to set up `danger` for your repository
+
+```
+danger init
+```
+
 ## Usage
 
 In CI run `bundle exec danger`.  This will look at your `Dangerfile` and provide some feedback based on that.
 
 ## DSL
 
-Danger :no_entry_sign:  | &nbsp; | &nbsp;
+&nbsp;  | &nbsp; | Danger :no_entry_sign:
 -------------: | ------------- | ----
 :sparkles: | `lines_of_code` | The total amount of lines of code in the diff
-:monorail: | `files_modified` |  The list of files modified
+:pencil2:  | `files_modified` |  The list of files modified
 :ship: | `files_added` | The list of files added
-:pencil2: | `files_removed` | The list of files removed
-:wrench: | `pr_title` | The title of the PR
-:thought_balloon: | `pr_body` | The body of the PR
-
-
-
-You can access more detailed information  by looking through:
-
-Danger :no_entry_sign:  | &nbsp; | &nbsp;
--------------: | ------------- | ----
-| :sparkles: |  `env.travis` | Details on the travis integration
-| :tophat: |`env.circle` |  Details on the circle integration
-| :octocat: | `env.github.pr_json` | The full JSON for the pull request
-| :ghost: | `env.git.diff` | The full [GitDiff](https://github.com/schacon/ruby-git/blob/master/lib/git/diff.rb) file for the diff.
+:recycle: | `files_removed` | The list of files removed
+:abc:  | `pr_title` | The title of the PR
+:book:  | `pr_body` | The body of the PR
+:busts_in_silhouette:  | `pr_author` | The author who submitted the PR
 
 You can then create a `Dangerfile` like the following:
 
@@ -52,6 +63,9 @@ end
 if lines_of_code > 50 && pr_title.include? "📱" == false
    fail("Needs testing on a Phone if change is non-trivial")
 end
+
+message("This pull request adds #{lines_of_code} new lines")
+warn("Author @#{pr_author} is not a contributor") unless ["KrauseFx", "orta"].include?(pr_author)
 ```
 
 ## Constraints
@@ -59,14 +73,20 @@ end
 * **GitHub** - Built with same-repo PRs in mind
 * **Git** - Built with master as the merge branch
 
-PRs welcome on these
+## Advanced
+
+You can access more detailed information by accessing the following variables
 
-## Development
+&nbsp; | Danger :no_entry_sign:
+------------- | ----
+`env.request_source.pr_json` | The full JSON for the pull request
+`env.scm.diff` | The full [GitDiff](https://github.com/schacon/ruby-git/blob/master/lib/git/diff.rb) file for the diff.
+`env.ci_source` | To get information like the repo slug or pull request ID
 
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
+## Special Thanks
 
-To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
+Thanks [@orta](https://twitter.com/orta) for starting this project
 
-## Contributing
+## License
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/orta/danger.
+> This project is open source under the MIT license, which means you have full access to the source code and can modify it to fit your own needs. 

data/bin/danger

@@ -0,0 +1,5 @@
+#!/usr/bin/env ruby
+$LOAD_PATH.push File.expand_path("../../lib", __FILE__)
+
+require 'danger'
+Danger::Runner.run ARGV

data/lib/assets/DangerfileTemplate

@@ -0,0 +1,15 @@
+puts "OK"
+puts "Lines"
+puts lines_of_code
+puts "Added"
+puts files_added
+puts "modified"
+puts files_modified
+
+puts ""
+puts pr_body
+puts ""
+puts pr_title
+
+warn("Some random warning")
+fail("Orta is not really orta")

data/lib/danger.rb

@@ -1,43 +1,26 @@
 require "danger/version"
 require "danger/dangerfile"
 require "danger/environment_manager"
+require "danger/runner"
+require "danger/init"
+require "danger/available_values"
 
-require 'claide'
-require 'colored'
+require "claide"
+require "colored"
+require "pathname"
 
-module Danger
-  class DangerRunner < CLAide::Command
-
-    self.description = 'Run the Dangerfile.'
-    self.command = 'danger'
-
-    def initialize(argv)
-      @dangerfile_path = "Dangerfile" if File.exist? "Dangerfile"
-      super
-    end
-
-    def validate!
-      super
-      unless @dangerfile_path
-        help! "Could not find a Dangerfile."
-      end
-    end
-
-    def run
-      dm = Dangerfile.new
-      dm.env = EnvironmentManager.new(ENV)
-      dm.env.fill_environment_vars
-      dm.update_from_env
-      dm.env.git.diff_for_folder(".")
-      dm.parse Pathname.new(@dangerfile_path)
+# Import all the Sources (CI, Request and SCM)
+Dir[File.expand_path('danger/*source/*.rb', File.dirname(__FILE__))].each do |file|
+  require file
+end
 
-      if dm.failures
-        puts "Uh Oh failed"
-        exit(1)
-      else
-        puts "The Danger has passed. Phew."
-      end
+module Danger
+  # @return [String] The path to the local gem directory
+  def self.gem_path
+    gem_name = "danger"
+    unless Gem::Specification.find_all_by_name(gem_name).any?
+      raise "Couldn't find gem directory for 'danger'"
     end
-
+    return Gem::Specification.find_by_name(gem_name).gem_dir
   end
 end

data/lib/danger/available_values.rb

@@ -0,0 +1,27 @@
+module Danger
+  # Defines all the values that should be available in someone's Dangerfile
+  class AvailableValues
+    def self.all
+      self.scm + self.request_source
+    end
+
+    def self.scm
+      [
+        :lines_of_code,
+        :files_modified,
+        :files_removed,
+        :files_added,
+        :deletions,
+        :insertions
+      ]
+    end
+
+    def self.request_source
+      [
+        :pr_title,
+        :pr_body,
+        :pr_author
+      ]
+    end
+  end
+end

data/lib/danger/ci_source/buildkite.rb

@@ -0,0 +1,22 @@
+# https://buildkite.com/docs/agent/osx
+# https://buildkite.com/docs/guides/environment-variables
+
+module Danger
+  module CISource
+    class Buildkite < CI
+      def self.validates?(env)
+        return !env["BUILDKITE"].nil?
+      end
+
+      def initialize(env)
+        repo = env["BUILDKITE_REPO"]
+        unless repo.nil?
+          repo_matches = repo.match(%r{([\/:])([^\/]+\/[^\/.]+)(?:.git)?$})
+          self.repo_slug = repo_matches[2] unless repo_matches.nil?
+        end
+
+        self.pull_request_id = env["BUILDKITE_PULL_REQUEST"]
+      end
+    end
+  end
+end

data/lib/danger/ci_source/ci_source.rb

@@ -0,0 +1,16 @@
+module Danger
+  module CISource
+    # "abstract" CI class
+    class CI
+      attr_accessor :repo_slug, :pull_request_id
+
+      def self.validates?(_env)
+        false
+      end
+
+      def initialize(_env)
+        raise "Subclass and overwrite initialize"
+      end
+    end
+  end
+end

data/lib/danger/ci_source/circle.rb

@@ -2,20 +2,18 @@
 require 'uri'
 
 module Danger
-  class CircleCI
+  module CISource
+    class CircleCI < CI
+      def self.validates?(env)
+        return !env["CIRCLE_BUILD_NUM"].nil? && !env["CI_PULL_REQUEST"].nil?
+      end
 
-    attr_accessor :repo_slug, :pull_request_id
-
-    def self.validates?(env)
-      return env["CIRCLE_BUILD_NUM"] != nil && ["CI_PULL_REQUEST"] != nil
-    end
-
-    def initialize(env)
-      paths = URI::parse(env["CI_PULL_REQUEST"]).path.split("/")
-      # the first one is an extra slash, ignore it
-      self.repo_slug = paths[1] + "/" + paths[2]
-      self.pull_request_id = paths[4]
+      def initialize(env)
+        paths = URI.parse(env["CI_PULL_REQUEST"]).path.split("/")
+        # the first one is an extra slash, ignore it
+        self.repo_slug = paths[1] + "/" + paths[2]
+        self.pull_request_id = paths[4]
+      end
     end
-
   end
 end

data/lib/danger/ci_source/travis.rb

@@ -2,18 +2,19 @@
 # http://docs.travis-ci.com/user/environment-variables/
 
 module Danger
-  class Travis
+  module CISource
+    class Travis < CI
+      def self.validates?(env)
+        return !env["HAS_JOSH_K_SEAL_OF_APPROVAL"].nil?
+      end
 
-    attr_accessor :repo_slug, :pull_request_id
-
-    def self.validates?(env)
-      return env["HAS_JOSH_K_SEAL_OF_APPROVAL"] != nil
-    end
-
-    def initialize(env)
-      self.repo_slug = env["TRAVIS_REPO_SLUG"]
-      self.pull_request_id = env["TRAVIS_PULL_REQUEST"]
+      def initialize(env)
+        self.repo_slug = env["TRAVIS_REPO_SLUG"]
+        # from https://docs.travis-ci.com/user/pull-requests, as otherwise it's "false"
+        if env["TRAVIS_PULL_REQUEST"].to_i > 0
+          self.pull_request_id = env["TRAVIS_PULL_REQUEST"]
+        end
+      end
     end
-
   end
 end

data/lib/danger/comment_generators/github.md.erb

@@ -0,0 +1,32 @@
+<% if @errors.count > 0 %>
+  &nbsp; | <%= @errors.count %> Error<%= "s" unless @errors.count == 1 %>
+  ------------- | ------------ <% @errors.each do |error| %>
+    :no_entry_sign: | <%= error %><% end %>
+<% else %>
+  :white_check_mark: | No errors found
+  ------------- | ------------ 
+<% end %>
+
+<% if @warnings.count > 0 %>
+  &nbsp; | <%= @warnings.count %> Warning<%= "s" unless @warnings.count == 1 %>
+  ------------- | ------------ <% @warnings.each do |warning| %>
+    :warning: | <%= warning %><% end %>
+<% else %>
+  :white_check_mark: | No warnings found
+  ------------- | ------------ 
+<% end %>
+
+<% if @messages.count > 0 %>
+  &nbsp; | <%= @messages.count %> Message<%= "s" unless @messages.count == 1 %>
+  ------------- | ------------ <% @messages.each do |message| %>
+    :book: | <%= message %><% end %>
+<% end %>
+
+<p align="right">
+  Generated by 
+    <a href="https://github.com/KrauseFx/danger">danger</a>
+  on 
+  <i><%= Time.now.strftime("%Y-%m-%d") %></i>
+</p>
+
+<% "" # the reason why we have the each in the same line is because we don't want a new line in the markdown file %>

data/lib/danger/{Dangerfile.rb → dangerfile.rb}

@@ -5,16 +5,12 @@ require 'danger/standard_error'
 
 module Danger
   class Dangerfile
-
     include Danger::Dangerfile::DSL
 
-    # The DSL includes a bunch of read only  attributes + docs
-    # we make them readwrite in here
-    attr_accessor :files_modified, :files_removed, :files_added, :pr_title, :pr_body
-    attr_accessor :env, :warnings, :failures
+    attr_accessor :env, :warnings, :errors, :messages
 
     # @return [Pathname] the path where the Dangerfile was loaded from. It is nil
-    #         if the podfile was generated programmatically.
+    #         if the Dangerfile was generated programmatically.
     #
     attr_accessor :defined_in_file
 
@@ -28,9 +24,7 @@ module Danger
     # Parses the file at a path, optionally takes the content of the file for DI
     #
     def parse(path, contents = nil)
-      warnings = [], failures =[]
-
-      contents ||= File.open(path, 'r:utf-8') { |f| f.read }
+      contents ||= File.open(path, 'r:utf-8', &:read)
 
       # Work around for Rubinius incomplete encoding in 1.9 mode
       if contents.respond_to?(:encoding) && contents.encoding.name != 'UTF-8'
@@ -39,12 +33,16 @@ module Danger
 
       if contents.tr!('“”‘’‛', %(""'''))
         # Changes have been made
-        puts        "Your #{path.basename} has had smart quotes sanitised. " \
+        puts "Your #{path.basename} has had smart quotes sanitised. " \
                     'To avoid issues in the future, you should not use ' \
                     'TextEdit for editing it. If you are not using TextEdit, ' \
                     'you should turn off smart quotes in your editor of choice.'.red
       end
 
+      if contents.include?("puts")
+        puts "You used `puts` in your Dangerfile. To print out text to GitHub use `message` instead"
+      end
+
       self.defined_in_file = path
       instance_eval do
         # rubocop:disable Lint/RescueException
@@ -59,14 +57,5 @@ module Danger
         # rubocop:enable Lint/RescueException
       end
     end
-
-    def update_from_env
-      self.files_modified = env.git.modified_files
-      self.files_removed = env.git.removed_files
-      self.files_added = env.git.added_files
-      self.pr_title = env.github.pr_title
-      self.pr_body = env.github.pr_body
-    end
-
   end
 end

data/lib/danger/dangerfile_dsl.rb

@@ -1,51 +1,27 @@
 module Danger
   class Dangerfile
     module DSL
-
-      public
-
       # @!group Enviroment
-      # @return [EnvironmentManager] Provides access to the raw Travis/Circle/GitHub
+      # @return [EnvironmentManager] Provides access to the raw Travis/Circle/Buildkite/GitHub
       # objects, which you can use to pull out extra bits of information. _Warning_
       # the api of these objects is **not** considered a part of the Dangerfile public
       # API, and is viable to change occasionally on the whims of developers.
 
       attr_reader :env
 
-      # @!group Code
-      # @return [Number] The total amount of lines of code in the diff
-      #
-      attr_reader :lines_of_code
-
-      # @return [Array of Strings] The list of files modified
-      #
-      attr_reader :files_modified
-
-      # @return [Array of Strings] The list of files removed
-      #
-      attr_reader :files_removed
-
-      # @return [Array of Strings] The list of files added
-      #
-      attr_reader :files_added
-
-      # @!group Pull Request Meta
-      # @return [String] The title of the PR
-      #
-      attr_reader :pr_title
-
-      # @return [String] The body of the PR
-      #
-      attr_reader :pr_body
-
+      def initialize
+        self.warnings = []
+        self.errors = []
+        self.messages = []
+      end
 
       # Declares a CI blocking error
       #
       # @param    [String] message
       #           The message to present to the user
       def fail(message)
-        self.failures << message
-        puts "fail #{fail}"
+        self.errors << message
+        puts "Raising error '#{message}'"
       end
 
       # Specifies a problem, but not critical
@@ -53,10 +29,34 @@ module Danger
       # @param    [String] message
       #           The message to present to the user
       def warn(message)
-        self.warnings << "message"
-        puts "warn #{fail}"
+        self.warnings << message
+        puts "Printing warning '#{message}'"
       end
 
+      # Print out a generate message on the PR
+      #
+      # @param    [String] message
+      #           The message to present to the user
+      def message(message)
+        self.messages << message
+        puts "Printing message '#{message}'"
+      end
+
+      def method_missing(method_sym, *_arguments, &_block)
+        unless AvailableValues.all.include?(method_sym)
+          raise "Unknown method '#{method_sym}', please check out the documentation for available variables".red
+        end
+
+        if AvailableValues.scm.include?(method_sym)
+          # SCM Source
+          return env.scm.send(method_sym)
+        end
+
+        if AvailableValues.request_source.include?(method_sym)
+          # Request Source
+          return env.request_source.send(method_sym)
+        end
+      end
     end
   end
 end

data/lib/danger/environment_manager.rb

@@ -1,24 +1,35 @@
-require "danger/ci_source/travis"
-require "danger/ci_source/circle"
+require "danger/ci_source/ci_source"
 require "danger/request_sources/github"
 
 module Danger
   class EnvironmentManager
-
-    attr_accessor :travis, :circle, :github, :git
+    attr_accessor :ci_source, :request_source, :scm
 
     def initialize(env)
-      self.travis = Travis.new(env) if Travis.validates?(env)
-      self.circle = CircleCI.new(env) if CircleCI.validates?(env)
-      raise "Could not find a CI source" unless self.travis || self.circle
+      CISource.constants.each do |symb|
+        c = CISource.const_get(symb)
+        next unless c.kind_of?(Class)
+        next unless c.validates?(env)
+
+        self.ci_source = c.new(env)
+        if self.ci_source.repo_slug and self.ci_source.pull_request_id
+          break
+        else
+          puts "Not a Pull Request - skipping `danger` run"
+          self.ci_source = nil
+          return nil
+        end
+      end
 
-      self.github = GitHub.new( travis || circle)
+      raise "Could not find a CI source".red unless self.ci_source
+
+      self.request_source = GitHub.new(self.ci_source) # for now
     end
 
     def fill_environment_vars
-      github.get_details
-      
-      self.git = GitRepo.new
+      request_source.fetch_details
+
+      self.scm = GitRepo.new # For now
     end
   end
 end

data/lib/danger/init.rb

@@ -0,0 +1,25 @@
+module Danger
+  class Init < Danger::Runner
+    self.description = 'Creates a Dangerfile.'
+    self.command = 'init'
+
+    def initialize(argv)
+      @dangerfile_path = "Dangerfile" if File.exist? "Dangerfile"
+      super
+    end
+
+    def validate!
+      if @dangerfile_path
+        help! "Found an existing Dangerfile."
+      end
+    end
+
+    def run
+      dir = Danger.gem_path
+
+      content = File.read(File.join(dir, "lib", "assets", "DangerfileTemplate"))
+      File.write("Dangerfile", content)
+      puts "Successfully created 'Dangerfile'"
+    end
+  end
+end

data/lib/danger/request_sources/github.rb

@@ -1,41 +1,97 @@
 require 'rest'
 require 'json'
+require 'base64'
+require 'octokit'
 
 module Danger
   class GitHub
-
     attr_accessor :ci_source, :pr_json
 
     def initialize(ci_source)
       self.ci_source = ci_source
     end
 
-    def api_url
-      "https://api.github.com/repos/#{ci_source.repo_slug}/pulls/#{ci_source.pull_request_id}"
+    def client
+      token = ENV["DANGER_GITHUB_API_TOKEN"]
+      raise "No API given, please provide one using `DANGER_GITHUB_API_TOKEN`" unless token
+
+      @client ||= Octokit::Client.new(
+        access_token: token
+      )
     end
 
-    def get_details
-      response = REST.get api_url
-      if response.ok?
-        self.pr_json = JSON.parse(response.body)
-      else
-        puts "Something went wrong getting GitHub details for #{api_url} - (#{response.status_code})"
-        puts response.body
-        raise "Could not get the pull request details from GitHub."
-      end
+    def fetch_details
+      self.pr_json = client.pull_request(ci_source.repo_slug, ci_source.pull_request_id)
     end
 
     def latest_pr_commit_ref
-      self.pr_json['base']['sha']
+      self.pr_json[:head][:sha]
     end
 
     def pr_title
-      self.pr_json['title']
+      self.pr_json[:title]
     end
 
     def pr_body
-      self.pr_json['body']
+      self.pr_json[:body]
+    end
+
+    def pr_author
+      self.pr_json[:user][:login]
+    end
+
+    # Sending data to GitHub
+    def update_pull_request!(warnings: nil, errors: nil, messages: nil)
+      # First, add a comment
+      body = generate_comment(warnings: warnings, errors: errors, messages: messages)
+      result = client.add_comment(ci_source.repo_slug, ci_source.pull_request_id, body)
+      delete_old_comment!(except: result[:id])
+
+      # Now, set the pull request status
+      submit_pull_request_status!(warnings: warnings,
+                                    errors: errors,
+                               details_url: result['html_url'])
+    end
+
+    def submit_pull_request_status!(warnings: nil, errors: nil, details_url: nil)
+      status = (errors.count == 0 ? 'success' : 'failure')
+      client.create_status(ci_source.repo_slug, latest_pr_commit_ref, status, {
+        description: generate_github_description(warnings: warnings, errors: errors),
+        context: "KrauseFx/danger",
+        target_url: details_url
+      })
     end
 
+    # Get rid of the previously posted comment, to only have the latest one
+    def delete_old_comment!(except: nil)
+      issues = client.issue_comments(ci_source.repo_slug, ci_source.pull_request_id)
+      issues.each do |issue|
+        next unless issue[:body].gsub(/\s+/, "").include?("Generatedby<ahref=")
+        next if issue[:id] == except
+        client.delete_comment(ci_source.repo_slug, issue[:id])
+      end
+    end
+
+    def generate_github_description(warnings: nil, errors: nil)
+      if errors.count > 0
+        "danger found errors"
+      elsif warnings.count > 0
+        "⚠️ danger found warnings, merge with caution"
+      else
+        "danger was successful"
+      end
+    end
+
+    def generate_comment(warnings: nil, errors: nil, messages: nil)
+      require 'erb'
+
+      @warnings = warnings
+      @errors = errors
+      @messages = messages
+
+      md_template = File.join(Danger.gem_path, "lib/danger/comment_generators/github.md.erb")
+      comment = ERB.new(File.read(md_template)).result(binding) # http://www.rrn.dk/rubys-erb-templating-system
+      return comment
+    end
   end
 end

data/lib/danger/runner.rb

@@ -0,0 +1,35 @@
+module Danger
+  class Runner < CLAide::Command
+    self.description = 'Run the Dangerfile.'
+    self.command = 'danger'
+
+    def initialize(argv)
+      @dangerfile_path = "Dangerfile" if File.exist? "Dangerfile"
+      super
+    end
+
+    def validate!
+      super
+      unless @dangerfile_path
+        help! "Could not find a Dangerfile."
+      end
+    end
+
+    def run
+      # The order of the following commands is *really* important
+      dm = Dangerfile.new
+      dm.env = EnvironmentManager.new(ENV)
+      return unless dm.env.ci_source # if it's not a PR
+      dm.env.fill_environment_vars
+      dm.env.scm.diff_for_folder(".")
+      dm.parse Pathname.new(@dangerfile_path)
+
+      post_results(dm)
+    end
+
+    def post_results(dm)
+      gh = dm.env.request_source
+      gh.update_pull_request!(warnings: dm.warnings, errors: dm.errors, messages: dm.messages)
+    end
+  end
+end

data/lib/danger/scm_source/{git.rb → git_repo.rb}

@@ -11,16 +11,16 @@ module Danger
       self.diff = g.diff(to, from)
     end
 
-    def modified_files
-      @diff.to_a.map { |d| d.path }
+    def files_modified
+      @diff.to_a.map(&:path)
     end
 
-    def removed_files
-      @diff.to_a.select { |d| d.type == "deleted" } .map { |d| d.path }
+    def files_removed
+      @diff.to_a.select { |d| d.type == "deleted" }.map(&:path)
     end
 
-    def added_files
-      @diff.to_a.select { |d| d.type == "new" } .map { |d| d.path }
+    def files_added
+      @diff.to_a.select { |d| d.type == "new" }.map(&:path)
     end
 
     def lines_of_code
@@ -34,6 +34,5 @@ module Danger
     def insertions
       @diff.insertions
     end
-
   end
 end

data/lib/danger/standard_error.rb

@@ -78,16 +78,16 @@ module Danger
 
         lines      = contents.lines
         indent     = ' #  '
-        indicator  = indent.gsub('#', '>')
+        indicator  = indent.tr('#', '>')
         first_line = (line_numer.zero?)
         last_line  = (line_numer == (lines.count - 1))
 
         m << "\n"
         m << "#{indent}from #{trace_line.gsub(/:in.*$/, '')}\n"
         m << "#{indent}-------------------------------------------\n"
-        m << "#{indent}#{    lines[line_numer - 1] }" unless first_line
-        m << "#{indicator}#{ lines[line_numer] }"
-        m << "#{indent}#{    lines[line_numer + 1] }" unless last_line
+        m << "#{indent}#{lines[line_numer - 1]}" unless first_line
+        m << "#{indicator}#{lines[line_numer]}"
+        m << "#{indent}#{lines[line_numer + 1]}" unless last_line
         m << "\n" unless m.end_with?("\n")
         m << "#{indent}-------------------------------------------\n"
       end

data/lib/danger/version.rb

@@ -1,3 +1,4 @@
 module Danger
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
+  DESCRIPTION = "Ensure your pull request is up to standard with a nice DSL"
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: danger
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Orta Therox
 - Felix Krause
 autorequire: 
-bindir: exe
+bindir: bin
 cert_chain: []
-date: 2015-09-29 00:00:00.000000000 Z
+date: 2016-01-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide
@@ -67,6 +67,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: octokit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -109,41 +123,81 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fastlane
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.49.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.49.0
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Create a Dangerfile to introspect your pull request in CI, makes it easy
   to enforce social conventions like changelogs and tests.
 email:
 - orta.therox@gmail.com
-- fastlane@krausefx.com
+- danger@krausefx.com
 executables:
 - danger
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".rspec"
-- ".travis.yml"
-- CHANGELOG.md
-- Gemfile
 - LICENSE
 - README.md
-- Rakefile
-- bin/console
-- bin/setup
-- danger.gemspec
-- example/Dangerfile
-- exe/danger
+- bin/danger
+- lib/assets/DangerfileTemplate
 - lib/danger.rb
-- lib/danger/Dangerfile.rb
+- lib/danger/available_values.rb
+- lib/danger/ci_source/buildkite.rb
+- lib/danger/ci_source/ci_source.rb
 - lib/danger/ci_source/circle.rb
 - lib/danger/ci_source/travis.rb
+- lib/danger/comment_generators/github.md.erb
+- lib/danger/dangerfile.rb
 - lib/danger/dangerfile_dsl.rb
 - lib/danger/environment_manager.rb
+- lib/danger/init.rb
 - lib/danger/request_sources/github.rb
-- lib/danger/scm_source/git.rb
+- lib/danger/runner.rb
+- lib/danger/scm_source/git_repo.rb
 - lib/danger/standard_error.rb
 - lib/danger/version.rb
 homepage: http://github.com/orta/danger
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -153,7 +207,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -161,8 +215,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
-summary: Ensure your pull request is up to standard with a nice DSL.
+summary: Ensure your pull request is up to standard with a nice DSL
 test_files: []

data/.gitignore

@@ -1,44 +0,0 @@
-/.bundle/
-/.yardoc
-/Gemfile.lock
-/_yardoc/
-/coverage/
-/doc/
-/pkg/
-/spec/reports/
-/tmp/
-*.gem
-*.rbc
-/.config
-/coverage/
-/InstalledFiles
-/pkg/
-/spec/reports/
-/test/tmp/
-/test/version_tmp/
-/tmp/
-
-## Specific to RubyMotion:
-.dat*
-.repl_history
-build/
-
-## Documentation cache and generated files:
-/.yardoc/
-/_yardoc/
-/doc/
-/rdoc/
-
-## Environment normalisation:
-/.bundle/
-/vendor/bundle
-/lib/bundler/man/
-
-# for a library or gem, you might want to ignore these files since the code is
-# intended to run in multiple environments; otherwise, check them in:
-# Gemfile.lock
-# .ruby-version
-# .ruby-gemset
-
-# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
-.rvmrc

data/.rspec

@@ -1,2 +0,0 @@
---format documentation
---color

data/.travis.yml

@@ -1,4 +0,0 @@
-language: ruby
-rvm:
-  - 2.1.3
-before_install: gem install bundler -v 1.10.6

data/CHANGELOG.md

@@ -1,7 +0,0 @@
-## 0.1
-
-* Parses a `Dangerfile` - orta
-* Gets GitHub details from Travis & CircleCI - orta
-* Gets PR details from GitHub - orta
-* Gets Git details from local Git - orta
-* Fails when you say it's failed in  the  Dangerfile - orta

data/Gemfile

@@ -1,6 +0,0 @@
-source 'https://rubygems.org'
-
-# Specify your gem's dependencies in danger.gemspec
-gemspec
-
-gem "pry"

data/Rakefile

@@ -1,6 +0,0 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-
-RSpec::Core::RakeTask.new(:spec)
-
-task :default => :spec

data/bin/console

@@ -1,11 +0,0 @@
-#!/usr/bin/env ruby
-
-require "bundler/setup"
-require "danger"
-
-# You can add fixtures and/or initialization code here to make experimenting
-# with your gem easier. You can also use a different console, if you like.
-
-# (If you use this, don't forget to add pry to your Gemfile!)
-require "pry"
-Pry.start

data/bin/setup

@@ -1,7 +0,0 @@
-#!/bin/bash
-set -euo pipefail
-IFS=$'\n\t'
-
-bundle install
-
-# Do any other automated setup that you need to do here

data/danger.gemspec

@@ -1,29 +0,0 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'danger/version'
-
-Gem::Specification.new do |spec|
-  spec.name          = "danger"
-  spec.version       = Danger::VERSION
-  spec.authors       = ["Orta Therox", "Felix Krause"]
-  spec.email         = ["orta.therox@gmail.com", "fastlane@krausefx.com"]
-
-  spec.summary       = 'Ensure your pull request is up to standard with a nice DSL.'
-  spec.description   = 'Create a Dangerfile to introspect your pull request in CI, makes it easy to enforce social conventions like changelogs and tests.'
-  spec.homepage      = "http://github.com/orta/danger"
-
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
-
-  spec.add_runtime_dependency 'claide', "~> 0.8"
-  spec.add_runtime_dependency 'git', "~> 1.2.9"
-  spec.add_runtime_dependency 'colored'
-  spec.add_runtime_dependency 'nap'
-
-  spec.add_development_dependency "bundler", "~> 1.10"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec"
-end

data/example/Dangerfile

@@ -1,2 +0,0 @@
-puts "Hi"
-puts lines_of_code

data/exe/danger

@@ -1,7 +0,0 @@
-#!/usr/bin/env ruby
-
-lib = File.expand_path(File.dirname(__FILE__) + '/../lib')
-$LOAD_PATH.unshift(lib) if File.directory?(lib) && !$LOAD_PATH.include?(lib)
-
-require 'danger'
-Danger::DangerRunner.run ARGV