danger 0.9.1 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e02bc78c88ffbdd9d2c7c7b4abd87939c2a76837
-  data.tar.gz: 173e0a2993c408b5b6e53c379239e97228839bc3
+  metadata.gz: 7b697f1591c4b33509e536da0cf20d7acaa61e8c
+  data.tar.gz: 0a9d995a79188ae53f543230fbd9a7007c1641b8
 SHA512:
-  metadata.gz: 001cd98d2bd528920fd6aafffea0aca00b41672e32e52588ba30f1eb2e9614cd246bf066f7ced68bbe6ec289f90e0c03774ea8e6d075de1ced383f20909a8854
-  data.tar.gz: 65669c564bcae4590b3bc384b5d3a96c16b74404e48d4489eaef4cc6d17a8a53270db8e5cfa3969579c3bac5176b47187de777f31012b92dcc52b660a5b59591
+  metadata.gz: 0a1350d0ce5fbfe8c22c27c04848243a589b07e39e6805ae15343259891153145f2aacc6bee0505101cc53052d35c44b83efb3e3e05733469f9eddb6d8850bd4
+  data.tar.gz: 0bcc00b71b54a8b4d23ac457feab9793d742c110fa55ee5997a584b7719adad5591fe4faa9f27804cff7c9253759aab6c0a63e63763825e7ea7ff0af56e19019

data/README.md

@@ -201,6 +201,16 @@ Danger will update the comment to cross it out. If you don't want this behavior,
 fail("PR needs labels", sticky: false) if pr_labels.empty?
 ```
 
+## Org-Wide Dangerfile
+
+To have consistent rules across the repos in your organisation, you can use a shared `Dangerfile`:
+
+1. create a repo on your GitHub organization called `danger` (or `Danger`)
+1. Add a `Dangerfile` to the root of that repo
+1. The new `Dangerfile` is ran after the current repo's `Dangerfile`
+
+The org `Dangerfile` will have access to all of the same plugins, and metadata. For an example, see [themoji/danger](https://github.com/Themoji/danger).
+
 ## Multiple Dangers
 
 If one Danger is not enough for you, you can run several ones on the same PR. Just use the `danger_id` param. For example: 

data/lib/danger.rb

@@ -4,6 +4,7 @@ require "danger/danger_core/environment_manager"
 require "danger/commands/runner"
 require "danger/plugin_support/plugin"
 require "danger/core_ext/string"
+require "danger/danger_core/executor"
 
 require "claide"
 require "colored"

data/lib/danger/ci_source/buildkite.rb

@@ -3,6 +3,7 @@
 
 module Danger
   module CISource
+    # https://buildkite.com
     class Buildkite < CI
       def self.validates?(env)
         return false unless env["BUILDKITE"]

data/lib/danger/ci_source/circle.rb

@@ -4,6 +4,7 @@ require "danger/ci_source/circle_api"
 
 module Danger
   module CISource
+    # https://circleci.com
     class CircleCI < CI
       def self.validates?(env)
         return false unless env["CIRCLE_BUILD_NUM"]

data/lib/danger/ci_source/drone.rb

@@ -2,6 +2,7 @@
 
 module Danger
   module CISource
+    # https://drone.io
     class Drone < CI
       def self.validates?(env)
         return false unless env["DRONE"]

data/lib/danger/ci_source/jenkins.rb

@@ -3,6 +3,7 @@
 
 module Danger
   module CISource
+    # https://jenkins-ci.org
     class Jenkins < CI
       def self.validates?(env)
         return false unless env["ghprbPullId"].to_i > 0

data/lib/danger/ci_source/local_git_repo.rb

@@ -5,6 +5,7 @@ require "uri"
 
 module Danger
   module CISource
+    # ignore
     class LocalGitRepo < CI
       attr_accessor :base_commit, :head_commit
 

data/lib/danger/ci_source/semaphore.rb

@@ -2,6 +2,7 @@
 
 module Danger
   module CISource
+    # https://semaphoreci.com
     class Semaphore < CI
       def self.validates?(env)
         return false unless env["SEMAPHORE"]

data/lib/danger/ci_source/surf.rb

@@ -2,6 +2,7 @@
 
 module Danger
   module CISource
+    # http://github.com/surf-build/surf
     class Surf < CI
       def self.validates?(env)
         return ["SURF_REPO", "SURF_NWO"].all? { |x| env[x] }

data/lib/danger/ci_source/teamcity.rb

@@ -1,5 +1,6 @@
 module Danger
   module CISource
+    # https://www.jetbrains.com/teamcity/
     class TeamCity < CI
       def self.validates?(env)
         env.key? "TEAMCITY_VERSION"

data/lib/danger/ci_source/travis.rb

@@ -3,6 +3,7 @@
 
 module Danger
   module CISource
+    # https://travis-ci.com
     class Travis < CI
       def self.validates?(env)
         return false unless env["HAS_JOSH_K_SEAL_OF_APPROVAL"]

data/lib/danger/ci_source/xcode_server.rb

@@ -2,6 +2,7 @@
 
 module Danger
   module CISource
+    # https://github.com/czechboy0/Buildasaur
     class XcodeServer < CI
       def self.validates?(env)
         return false unless env["XCS_BOT_NAME"]

data/lib/danger/commands/local.rb

@@ -1,3 +1,8 @@
+require "danger/commands/local_helpers/http_cache"
+require "faraday/http_cache"
+require "octokit"
+require "tmpdir"
+
 module Danger
   class Local < Runner
     self.summary = "Run the Dangerfile locally."
@@ -6,12 +11,14 @@ module Danger
     def initialize(argv)
       @dangerfile_path = "Dangerfile" if File.exist? "Dangerfile"
       @pr_num = argv.option("use-merged-pr")
+      @clear_http_cache = argv.flag?("clear-http-cache", false)
       super
     end
 
     def self.options
       [
-        ["--use-merged-pr=[#id]", "The ID of an already merged PR inside your history to use as a reference for the local run."]
+        ["--use-merged-pr=[#id]", "The ID of an already merged PR inside your history to use as a reference for the local run."],
+        ["--clear-http-cache", "Clear the local http cache before running Danger locally."]
       ].concat(super)
     end
 
@@ -26,6 +33,15 @@ module Danger
       ENV["DANGER_USE_LOCAL_GIT"] = "YES"
       ENV["LOCAL_GIT_PR_ID"] = @pr_num if @pr_num
 
+      # setup caching for Github calls to hitting the API rate limit too quickly
+      cache_file = File.join(ENV["DANGER_TMPDIR"] || Dir.tmpdir, "danger_local_cache")
+      cache = HTTPCache.new(cache_file, clear_cache: @clear_http_cache)
+      Octokit.middleware = Faraday::Builder.new do |builder|
+        builder.use Faraday::HttpCache, store: cache, serializer: Marshal, shared_cache: false
+        builder.use Octokit::Response::RaiseError
+        builder.adapter Faraday.default_adapter
+      end
+
       env = EnvironmentManager.new(ENV)
       dm = Dangerfile.new(env, cork)
       dm.init_plugins
@@ -64,11 +80,23 @@ module Danger
         dm.env.fill_environment_vars
         dm.env.ensure_danger_branches_are_setup
         dm.env.scm.diff_for_folder(".", from: Danger::EnvironmentManager.danger_base_branch, to: Danger::EnvironmentManager.danger_head_branch)
+
         dm.parse(Pathname.new(@dangerfile_path))
+        check_and_run_org_dangerfile(dm)
+
         dm.print_results
       ensure
         dm.env.clean_up
       end
     end
+
+    # Check to see if there's a Dangerfile in the organisation, and run it if so
+    def check_and_run_org_dangerfile(dm)
+      if dm.env.request_source.organisation && !dm.env.request_source.danger_repo? && (danger_repo = dm.env.request_source.fetch_danger_repo)
+        url = dm.env.request_source.file_url(repository: danger_repo.name, path: "Dangerfile")
+        path = dm.plugin.download(url)
+        dm.parse(Pathname.new(path))
+      end
+    end
   end
 end

data/lib/danger/commands/local_helpers/http_cache.rb

@@ -0,0 +1,36 @@
+require "pstore"
+
+module Danger
+  class HTTPCache
+    attr_reader :expires_in
+    def initialize(cache_file = nil, options = {})
+      File.delete(cache_file) if options[:clear_cache]
+      @store = PStore.new(cache_file)
+      @expires_in = options[:expires_in] || 300 # 5 minutes
+    end
+
+    def read(key)
+      @store.transaction do
+        entry = @store[key]
+        return nil unless entry
+        return entry[:value] unless entry_has_expired(entry, @expires_in)
+        @store.delete key
+        return nil
+      end
+    end
+
+    def delete(key)
+      @store.transaction { @store.delete key }
+    end
+
+    def write(key, value)
+      @store.transaction do
+        @store[key] = { updated_at: Time.now.to_i, value: value }
+      end
+    end
+
+    def entry_has_expired(entry, ttl)
+      Time.now.to_i > entry[:updated_at].to_i + ttl.to_i
+    end
+  end
+end

data/lib/danger/commands/plugins/plugin_lint.rb

@@ -32,7 +32,7 @@ module Danger
       file_resolver = PluginFileResolver.new(@refs)
       paths = file_resolver.resolve_to_paths
 
-      parser = PluginParser.new(paths)
+      parser = PluginParser.new(paths, verbose: true)
       parser.parse
       json = parser.to_json
 

data/lib/danger/commands/runner.rb

@@ -16,14 +16,14 @@ module Danger
     require "danger/commands/plugins/plugin_json"
     require "danger/commands/plugins/plugin_readme"
 
+    attr_accessor :cork
+
     self.summary = "Run the Dangerfile."
     self.command = "danger"
     self.version = Danger::VERSION
 
     self.plugin_prefixes = %w(claide danger)
 
-    attr_accessor :cork
-
     def initialize(argv)
       dangerfile = argv.option("dangerfile", "Dangerfile")
       @dangerfile_path = dangerfile if File.exist? dangerfile
@@ -52,53 +52,10 @@ module Danger
     end
 
     def run
-      env = EnvironmentManager.new(ENV)
-      dm = Dangerfile.new(env, cork)
-
-      if dm.env.pr?
-        dm.verbose = verbose
-        dm.init_plugins
-
-        dm.env.fill_environment_vars
-
-        begin
-          dm.env.ensure_danger_branches_are_setup
-
-          # Offer the chance for a user to specify a branch through the command line
-          ci_base = @base || EnvironmentManager.danger_base_branch
-          ci_head = @head || EnvironmentManager.danger_head_branch
-          dm.env.scm.diff_for_folder(".", from: ci_base, to: ci_head)
-
-          dm.parse Pathname.new(@dangerfile_path)
-
-          post_results dm
-          dm.print_results
-        ensure
-          dm.env.clean_up
-        end
-      else
-        cork.puts "Not a Pull Request - skipping `danger` run"
-      end
-    end
-
-    def post_results(dm)
-      gh = dm.env.request_source
-      violations = dm.violation_report
-      status = dm.status_report
-
-      gh.update_pull_request!(warnings: violations[:warnings], errors: violations[:errors], messages: violations[:messages], markdowns: status[:markdowns], danger_id: @danger_id)
-    end
-
-    def self.report_error(exception)
-      raise exception if exception.kind_of?(SystemExit)
-      message = "#{exception.message.red} (#{exception.class.to_s.yellow})"
-      if exception.backtrace
-        danger_lib = File.expand_path("../../..", __FILE__)
-        message << "\n\t" << exception.backtrace.reverse_each.
-                             drop_while { |bt| !bt.start_with?(danger_lib) }.reverse.
-                             join("\n\t")
-      end
-      abort(message)
+      Executor.new.run(base: @base,
+                       head: @head,
+                       dangerfile_path: @dangerfile_path,
+                       danger_id: @danger_id)
     end
   end
 end

data/lib/danger/danger_core/executor.rb

@@ -0,0 +1,55 @@
+module Danger
+  class Executor
+    def run(env: nil,
+            dm: nil,
+            cork: nil,
+            base: nil,
+            head: nil,
+            dangerfile_path: nil,
+            danger_id: nil)
+
+      cork ||= Cork::Board.new(silent: false,
+                              verbose: false)
+      env ||= EnvironmentManager.new(ENV)
+      dm ||= Dangerfile.new(env, cork)
+
+      if dm.env.pr?
+        dm.init_plugins
+
+        dm.env.fill_environment_vars
+
+        begin
+          dm.env.ensure_danger_branches_are_setup
+
+          # Offer the chance for a user to specify a branch through the command line
+          ci_base = base || EnvironmentManager.danger_base_branch
+          ci_head = head || EnvironmentManager.danger_head_branch
+          dm.env.scm.diff_for_folder(".", from: ci_base, to: ci_head)
+
+          dm.parse(Pathname.new(dangerfile_path))
+
+          if dm.env.request_source.organisation && !dm.env.request_source.danger_repo? && (danger_repo = dm.env.request_source.fetch_danger_repo)
+            url = dm.env.request_source.file_url(repository: danger_repo.name, path: "Dangerfile")
+            path = dm.plugin.download(url)
+            dm.parse(Pathname.new(path))
+          end
+
+          post_results(dm, danger_id)
+          dm.print_results
+        ensure
+          dm.env.clean_up
+        end
+      else
+        cork.puts "Not a Pull Request - skipping `danger` run"
+      end
+    end
+
+    def post_results(dm, danger_id)
+      gh = dm.env.request_source
+      violations = dm.violation_report
+      status = dm.status_report
+
+      gh.update_pull_request!(warnings: violations[:warnings], errors: violations[:errors], messages: violations[:messages], markdowns: status[:markdowns], danger_id: danger_id)
+    end
+  end
+end

data/lib/danger/danger_core/plugins/dangerfile_git_plugin.rb

@@ -1,6 +1,7 @@
 require "danger/plugin_support/plugin"
 require "danger/core_ext/file_list"
 
+# Danger
 module Danger
   # Handles interacting with git inside a Dangerfile. Providing access to files that have changed, and useful statistics. Also provides
   # access to the commits in the form of [Git::Log](https://github.com/schacon/ruby-git/blob/master/lib/git/log.rb) objects.
@@ -30,6 +31,9 @@ module Danger
   # @tags core, git
 
   class DangerfileGitPlugin < Plugin
+    # The instance name used in the Dangerfile
+    # @return [String]
+    #
     def self.instance_name
       "git"
     end

data/lib/danger/danger_core/plugins/dangerfile_github_plugin.rb

@@ -33,6 +33,9 @@ module Danger
       @github = dangerfile.env.request_source
     end
 
+    # The instance name used in the Dangerfile
+    # @return [String]
+    #
     def self.instance_name
       "github"
     end
@@ -87,7 +90,7 @@ module Danger
 
     # @!group PR Commit Metadata
     # The base commit to which the PR is going to be merged as a parent.
-    # @return String
+    # @return [String]
     #
     def base_commit
       pr_json[:base][:sha]

data/lib/danger/danger_core/plugins/dangerfile_import_plugin.rb

@@ -24,6 +24,9 @@ module Danger
   # @tags core, plugins
 
   class DangerfileImportPlugin < Plugin
+    # The instance name used in the Dangerfile
+    # @return [String]
+    #
     def self.instance_name
       "plugin"
     end
@@ -31,32 +34,33 @@ module Danger
     # @!group Plugins
     # Download a local or remote plugin and use it inside the Dangerfile.
     #
-    # @param    [String] path
+    # @param    [String] path_or_url
     #           a local path or a https URL to the Ruby file to import
     #           a danger plugin from.
-    # @return [void]
-
-    def import(path)
-      raise "`import` requires a string" unless path.kind_of?(String)
-      path += ".rb" unless path.end_with?(".rb")
+    # @return   [void]
+    #
+    def import(path_or_url)
+      raise "`import` requires a string" unless path_or_url.kind_of?(String)
 
-      if path.start_with?("http")
-        import_url(path)
+      if path_or_url.start_with?("http")
+        import_url(path_or_url)
       else
-        import_local(path)
+        import_local(path_or_url)
       end
     end
 
-    private
-
     # @!group Plugins
-    # Download a remote plugin and use it locally.
+    # Download a local or remote plugin or Dangerfile
+    # This method will not import the file for you, use plugin.import instead
     #
-    # @param    [String] url
-    #           https URL to the Ruby file to use
-    # @return [void]
-    def import_url(url)
-      raise "URL is not https, for security reasons `danger` only supports encrypted requests" unless url.start_with?("https://")
+    # @param    [String] path_or_url
+    #           a local path or a https URL to the Ruby file to import
+    #           a danger plugin from.
+    # @return [String] The path to the downloaded Ruby file
+    #
+    def download(path_or_url)
+      raise "`download` requires a string" unless path_or_url.kind_of?(String)
+      raise "URL is not https, for security reasons `danger` only supports encrypted requests" if URI.parse(path_or_url).scheme != "https"
 
       require "tmpdir"
       require "faraday"
@@ -64,13 +68,24 @@ module Danger
       @http_client ||= Faraday.new do |b|
         b.adapter :net_http
       end
-      content = @http_client.get(url)
+      content = @http_client.get(path_or_url)
 
-      Dir.mktmpdir do |dir|
-        path = File.join(dir, "temporary_remote_action.rb")
-        File.write(path, content.body)
-        import_local(path)
-      end
+      path = File.join(Dir.mktmpdir, "temporary_danger.rb")
+      File.write(path, content.body)
+      return path
+    end
+
+    private
+
+    # @!group Plugins
+    # Download a remote plugin and use it locally.
+    #
+    # @param    [String] url
+    #           https URL to the Ruby file to use
+    # @return [void]
+    def import_url(url)
+      path = download(url)
+      import_local(path)
     end
 
     # @!group Plugins

data/lib/danger/danger_core/plugins/dangerfile_messaging_plugin.rb

@@ -53,6 +53,9 @@ module Danger
       @markdowns = []
     end
 
+    # The instance name used in the Dangerfile
+    # @return [String]
+    #
     def self.instance_name
       "messaging"
     end
@@ -62,6 +65,8 @@ module Danger
     #
     # @param    [String] message
     #           The markdown based message to be printed below the table
+    # @return   [void]
+    #
     def markdown(message)
       @markdowns << message
     end
@@ -69,10 +74,13 @@ module Danger
     # @!group Core
     # Print out a generate message on the PR
     #
-    # @param    [String] message The message to present to the user
+    # @param    [String] message
+    #           The message to present to the user
     # @param    [Boolean] sticky
     #           Whether the message should be kept after it was fixed,
     #           defaults to `true`.
+    # @return   [void]
+    #
     def message(message, sticky: true)
       @messages << Violation.new(message, sticky)
     end
@@ -80,10 +88,13 @@ module Danger
     # @!group Core
     # Specifies a problem, but not critical
     #
-    # @param    [String] message The message to present to the user
+    # @param    [String] message
+    #           The message to present to the user
     # @param    [Boolean] sticky
     #           Whether the message should be kept after it was fixed,
     #           defaults to `true`.
+    # @return   [void]
+    #
     def warn(message, sticky: true)
       return if should_ignore_violation(message)
       @warnings << Violation.new(message, sticky)
@@ -97,6 +108,8 @@ module Danger
     # @param    [Boolean] sticky
     #           Whether the message should be kept after it was fixed,
     #           defaults to `true`.
+    # @return   [void]
+    #
     def fail(message, sticky: true)
       return if should_ignore_violation(message)
       @errors << Violation.new(message, sticky)
@@ -106,7 +119,8 @@ module Danger
     # A list of all messages passed to Danger, including
     # the markdowns.
     #
-    # @return Hash
+    # @visibility hidden
+    # @return     [Hash]
     def status_report
       {
         errors: @errors.map(&:message).clone.freeze,
@@ -121,7 +135,7 @@ module Danger
     # anticipate users of Danger needing to use this.
     #
     # @visibility hidden
-    # @return Hash
+    # @return     [Hash]
     def violation_report
       {
         errors: @errors.clone.freeze,

data/lib/danger/plugin_support/plugin_linter.rb

@@ -51,7 +51,7 @@ module Danger
     # Did the linter pass/fail?
     #
     def failed?
-      errors.empty? == false
+      errors.count > 0
     end
 
     # Prints a summary of the errors and warnings.
@@ -59,15 +59,16 @@ module Danger
     def print_summary(ui)
       # Print whether it passed/failed at the top
       if failed?
-        ui.notice "Passed\n"
-      else
         ui.puts "\n[!] Failed\n".red
+      else
+        ui.notice "Passed"
       end
 
       # A generic proc to handle the similarities between
       # erros and warnings.
       do_rules = proc do |name, rules|
         unless rules.empty?
+          ui.puts ""
           ui.section(name.bold) do
             rules.each do |rule|
               title = rule.title.bold + " - #{rule.object_applied_to}"
@@ -111,7 +112,7 @@ module Danger
         Rule.new(:error, 40..41, "Description", "You should include a description for your method.", proc do |json|
           json[:body_md] && json[:body_md].empty?
         end),
-        Rule.new(:warning, 43..45, "Params", "If the function has no useful return value, use ` @return  [void]`.", proc do |json|
+        Rule.new(:warning, 43..45, "Params", "You should give a 'type' for the param, yes, ruby is duck-typey but it's useful for newbies to the language, use `@param [Type] name`.", proc do |json|
           json[:param_couplets] && json[:param_couplets].flat_map(&:values).include?(nil)
         end),
         Rule.new(:warning, 46, "Return Type", "If the function has no useful return value, use ` @return  [void]` - this will be ignored by documentation generators.", proc do |json|

data/lib/danger/plugin_support/plugin_parser.rb

@@ -34,9 +34,11 @@ module Danger
   class PluginParser
     attr_accessor :registry
 
-    def initialize(paths)
+    def initialize(paths, verbose = false)
       raise "Path cannot be empty" if paths.empty?
 
+      setup_yard(verbose)
+
       if paths.kind_of? String
         @paths = [File.expand_path(paths)]
       else
@@ -44,12 +46,20 @@ module Danger
       end
     end
 
-    def parse
+    def setup_yard(verbose)
       require 'yard'
-      # could this go in a singleton-y place instead?
-      # like class initialize?
+
+      # Unless specifically asked, don't output anything.
+      unless verbose
+        YARD::Logger.instance.level = YARD::Logger::FATAL
+      end
+
+      # Add some of our custom tags
       YARD::Tags::Library.define_tag('tags', :tags)
       YARD::Tags::Library.define_tag('availablity', :availablity)
+    end
+
+    def parse
       files = ["lib/danger/plugin_support/plugin.rb"] + @paths
 
       # This turns on YARD debugging
@@ -153,7 +163,7 @@ module Danger
         attribute_meths = klass.attributes[:instance].values.map(&:values).flatten
 
         methods = klass.meths - klass.inherited_meths - attribute_meths
-        usable_methods = methods.select { |m| m.visibility == :public }.reject { |m| m.name == :initialize || m.name == :insance_name }
+        usable_methods = methods.select { |m| m.visibility == :public }.reject { |m| m.name == :initialize || m.name == :instance_name }
 
         {
           name: klass.name.to_s,

data/lib/danger/request_source/github.rb

@@ -128,7 +128,12 @@ module Danger
           # use a read-only GitHub account
           if errors.count > 0
             # We need to fail the actual build here
-            abort("\nDanger has failed this build. \nFound #{'error'.danger_pluralize(errors.count)} and I don't have write access to the PR set a PR status.")
+            is_private = pr_json[:base][:repo][:private]
+            if is_private
+              abort("\nDanger has failed this build. \nFound #{'error'.danger_pluralize(errors.count)} and I don't have write access to the PR set a PR status.")
+            else
+              abort("\nDanger has failed this build. \nFound #{'error'.danger_pluralize(errors.count)}.")
+            end
           else
             puts message
           end
@@ -222,14 +227,56 @@ module Danger
       end
 
       def parse_tables_from_comment(comment)
-        comment.split('</table>')
+        comment.split("</table>")
       end
 
       def process_markdown(violation)
         html = markdown_parser.render(violation.message)
-        match = html.match(%r{^<p>(.*)</p>$})
-        message = match.nil? ? html : match.captures.first
-        Violation.new(message, violation.sticky)
+        # Remove the outer `<p>`, the -5 represents a newline + `</p>`
+        html = html[3...-5] if html.start_with? "<p>"
+        Violation.new(html, violation.sticky)
+      end
+
+      # @return [String] The organisation name, is nil if it can't be detected
+      def organisation
+        matched = self.issue_json[:repository_url].match(%r{repos\/(.*)\/})
+        return matched[1] if matched && matched[1]
+      rescue
+        nil
+      end
+
+      # @return [Hash] with the information about the repo
+      #   returns nil if the repo is not available
+      def fetch_repository(organisation: nil, repository: nil)
+        organisation ||= self.organisation
+        repository ||= self.ci_source.repo_slug.split("/").last
+        self.client.repo("#{organisation}/#{repository}")
+      rescue Octokit::NotFound
+        nil # repo doesn't exist
+      end
+
+      # @return [Hash] with the information about the repo.
+      #   This will automatically detect if the repo is capitalised
+      #   returns nil if there is no danger repo
+      def fetch_danger_repo(organisation: nil)
+        data = nil
+        data ||= fetch_repository(organisation: organisation, repository: DANGER_REPO_NAME.downcase)
+        data ||= fetch_repository(organisation: organisation, repository: DANGER_REPO_NAME.capitalize)
+        data
+      end
+
+      # @return [Bool] is this repo the danger repo of the org?
+      def danger_repo?(organisation: nil, repository: nil)
+        repo = fetch_repository(organisation: organisation, repository: repository)
+        repo[:name].casecmp(DANGER_REPO_NAME).zero?
+      rescue
+        false
+      end
+
+      # @return [String] A URL to the specific file, ready to be downloaded
+      def file_url(organisation: nil, repository: nil, branch: 'master', path: nil)
+        organisation ||= self.organisation
+        "https://raw.githubusercontent.com/#{organisation}/#{repository}/#{branch}/#{path}"
       end
     end
   end

data/lib/danger/request_source/request_source.rb

@@ -1,6 +1,8 @@
 module Danger
   module RequestSources
     class RequestSource
+      DANGER_REPO_NAME = "danger".freeze
+
       attr_accessor :ci_source, :environment, :scm, :host, :ignored_violations
 
       def self.inherited(child_class)
@@ -43,6 +45,26 @@ module Danger
       def fetch_details
         raise "Subclass and overwrite initialize"
       end
+
+      def organisation
+        raise "Subclass and overwrite organisation"
+      end
+
+      def fetch_repository(_organisation: nil, _repository: nil)
+        raise "Subclass and overwrite fetch_repository"
+      end
+
+      def fetch_danger_repo(_organisation: nil)
+        raise "Subclass and overwrite fetch_danger_repo"
+      end
+
+      def danger_repo?(_organisation: nil, _repository: nil)
+        raise "Subclass and overwrite danger_repo?"
+      end
+
+      def file_url(_organisation: nil, _repository: nil, _branch: "master", _path: nil)
+        raise "Subclass and overwrite file_url"
+      end
     end
   end
 end

data/lib/danger/version.rb

@@ -1,4 +1,4 @@
 module Danger
-  VERSION = "0.9.1".freeze
+  VERSION = "0.10.0".freeze
   DESCRIPTION = "Automate your PR etiquette.".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.10.0
 platform: ruby
 authors:
 - Orta Therox
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-14 00:00:00.000000000 Z
+date: 2016-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide
@@ -81,6 +81,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: faraday-http-cache
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: octokit
   requirement: !ruby/object:Gem::Requirement
@@ -322,6 +336,7 @@ files:
 - lib/danger/commands/init.rb
 - lib/danger/commands/init_helpers/interviewer.rb
 - lib/danger/commands/local.rb
+- lib/danger/commands/local_helpers/http_cache.rb
 - lib/danger/commands/plugins/plugin_json.rb
 - lib/danger/commands/plugins/plugin_lint.rb
 - lib/danger/commands/plugins/plugin_readme.rb
@@ -332,6 +347,7 @@ files:
 - lib/danger/danger_core/dangerfile.rb
 - lib/danger/danger_core/dangerfile_dsl.rb
 - lib/danger/danger_core/environment_manager.rb
+- lib/danger/danger_core/executor.rb
 - lib/danger/danger_core/plugins/dangerfile_git_plugin.rb
 - lib/danger/danger_core/plugins/dangerfile_github_plugin.rb
 - lib/danger/danger_core/plugins/dangerfile_import_plugin.rb
@@ -367,7 +383,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Automate your PR etiquette.