danger 3.3.0 → 3.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7e10fb247dbfbbf65bddac255ef51fbde2886ec7
-  data.tar.gz: 26e8a81ba580a77b844ee956875974dc52605e41
+  metadata.gz: 7c097c9080ca08cc97450e41454aea6db1634169
+  data.tar.gz: b7c32ddada66f84e92260dd1290e81f8e6619faf
 SHA512:
-  metadata.gz: 5af8334759ab6fd99078bdc24b25723f21266fc2a3ef603c66ea0940576f7b2228740aead12e1d6198afce6ce1ce46fb2f074d4b111d615f693bb29bb4eda08a
-  data.tar.gz: a9248512d06c5c37cbcf45f2d29ebb8ff02f325e8609c540b9eb05629fb0a01ac81fce622a198a7b13fc76fe3f04c1d879312f915a7480f44bf6b91db94c737b
+  metadata.gz: a20361f8bfd84a9ee0b4864b5379ea6e2c546d6757f237790c9a1246c0a1236f38173c72e46ba8ff2d239c03c228605f73aa62396d84a7dbe7357d9a9f8897ce
+  data.tar.gz: f869abaa8c464b490a475f3d61908e5fbad1e857270639c2fe29dbbbab510b67da2c27c544761c33e889bbff9e90a21362396fe991930fd4c9d30771bf0d9050

data/lib/danger/ci_source/bitrise.rb

@@ -1,4 +1,6 @@
 # http://devcenter.bitrise.io/docs/available-environment-variables
+require "danger/request_sources/github"
+require "danger/request_sources/gitlab"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/buildkite.rb

@@ -1,5 +1,7 @@
 # https://buildkite.com/docs/agent/osx
 # https://buildkite.com/docs/guides/environment-variables
+require "danger/request_sources/github"
+require "danger/request_sources/gitlab"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/circle.rb

@@ -1,6 +1,7 @@
 # https://circleci.com/docs/environment-variables
 require "uri"
 require "danger/ci_source/circle_api"
+require "danger/request_sources/github"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/drone.rb

@@ -1,4 +1,6 @@
 # http://readme.drone.io/usage/variables/
+require "danger/request_sources/github"
+require "danger/request_sources/gitlab"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/gitlab_ci.rb

@@ -1,5 +1,6 @@
 # http://docs.gitlab.com/ce/ci/variables/README.html
 require "uri"
+require "danger/request_sources/gitlab"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/jenkins.rb

@@ -1,5 +1,9 @@
 # https://wiki.jenkins-ci.org/display/JENKINS/Building+a+software+project#Buildingasoftwareproject-JenkinsSetEnvironmentVariables
 # https://wiki.jenkins-ci.org/display/JENKINS/GitHub+pull+request+builder+plugin
+require "danger/request_sources/github"
+require "danger/request_sources/gitlab"
+require "danger/request_sources/bitbucket_server"
+require "danger/request_sources/bitbucket_cloud"
 
 module Danger
   # https://jenkins-ci.org
@@ -38,7 +42,14 @@ module Danger
     end
 
     def supported_request_sources
-      @supported_request_sources ||= [Danger::RequestSources::GitHub, Danger::RequestSources::GitLab, Danger::RequestSources::BitbucketServer, Danger::RequestSources::BitbucketCloud]
+      @supported_request_sources ||= begin
+        [
+          Danger::RequestSources::GitHub,
+          Danger::RequestSources::GitLab,
+          Danger::RequestSources::BitbucketServer,
+          Danger::RequestSources::BitbucketCloud
+        ]
+      end
     end
 
     def initialize(env)

data/lib/danger/ci_source/local_git_repo.rb

@@ -2,6 +2,9 @@
 
 require "git"
 require "uri"
+require "danger/ci_source/support/remote_finder"
+require "danger/ci_source/support/merged_pull_request_finder"
+require "danger/request_sources/github"
 
 module Danger
   # ignore
@@ -28,39 +31,29 @@ module Danger
       @supported_request_sources ||= [Danger::RequestSources::GitHub]
     end
 
-    def initialize(env = {})
-      github_host = env["DANGER_GITHUB_HOST"] || "github.com"
-
-      # get the remote URL
-      remote = run_git("remote show origin -n").lines.grep(/Fetch URL/)[0].split(": ", 2)[1]
-      if remote
-        remote_url_matches = remote.match(%r{#{Regexp.escape github_host}(:|/)(?<repo_slug>.+/.+?)(?:\.git)?$})
-        if !remote_url_matches.nil? and remote_url_matches["repo_slug"]
-          self.repo_slug = remote_url_matches["repo_slug"]
-        else
-          puts "Danger local requires a repository hosted on GitHub.com or GitHub Enterprise."
-        end
-      end
-
-      specific_pr = env["LOCAL_GIT_PR_ID"]
-      pr_ref = specific_pr ? "##{specific_pr}" : ""
-      pr_command = "log --merges --oneline"
+    def print_repo_slug_warning
+      puts "Danger local requires a repository hosted on GitHub.com or GitHub Enterprise.".freeze
+    end
 
-      # get the most recent PR merge
-      pr_merge = run_git(pr_command.strip).lines.grep(Regexp.new("Merge pull request " + pr_ref))[0]
+    def parents(sha)
+      @parents ||= run_git("rev-list --parents -n 1 #{sha}").strip.split(" ".freeze)
+    end
 
-      if pr_merge.to_s.empty?
-        if specific_pr
-          raise "Could not find the pull request (#{specific_pr}) inside the git history for this repo."
-        else
-          raise "No recent pull requests found for this repo, danger requires at least one PR for the local mode."
-        end
-      end
-      self.pull_request_id = pr_merge.match("#([0-9]+)")[1]
-      sha = pr_merge.split(" ")[0]
-      parents = run_git("rev-list --parents -n 1 #{sha}").strip.split(" ")
-      self.base_commit = parents[0]
-      self.head_commit = parents[1]
+    def initialize(env = {})
+      repo_slug = RemoteFinder.new(
+        env["DANGER_GITHUB_HOST"] || "github.com".freeze,
+        run_git("remote show origin -n".freeze)
+      ).call
+
+      pull_request_id, sha = MergedPullRequestFinder.new(
+        env["LOCAL_GIT_PR_ID"] || "",
+        run_git("log --oneline -50".freeze)
+      ).call
+
+      self.repo_slug = repo_slug ? repo_slug : print_repo_slug_warning
+      self.pull_request_id = pull_request_id
+      self.base_commit = parents(sha)[0]
+      self.head_commit = parents(sha)[1]
     end
   end
 end

data/lib/danger/ci_source/semaphore.rb

@@ -1,4 +1,5 @@
 # https://semaphoreci.com/docs/available-environment-variables.html
+require "danger/request_sources/github"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/support/merged_pull_request_finder.rb

@@ -0,0 +1,66 @@
+module Danger
+  class MergedPullRequestFinder
+    def initialize(specific_pull_request_id, git_logs)
+      @specific_pull_request_id = specific_pull_request_id
+      @git_logs = git_logs
+    end
+
+    def call
+      check_if_any_merged_pull_request!
+
+      [merged_pull_request_id, merged_pull_request_sha]
+    end
+
+    private
+
+    attr_reader :specific_pull_request_id, :git_logs
+
+    # @return [String] "#42"
+    def pull_request_ref
+      specific_pull_request_id ? "##{specific_pull_request_id}" : "".freeze
+    end
+
+    # @return [String] Log line of format: "Merge pull request #42"
+    def most_recent_merged_pull_request
+      @most_recent_merged_pull_request ||= begin
+        git_logs.lines.grep(/Merge pull request #{pull_request_ref}/)[0]
+      end
+    end
+
+    # @return [String] Log line of format: "description (#42)"
+    def most_recent_squash_and_merged_pull_request
+      @most_recent_squash_and_merged_pull_request ||= begin
+        git_logs.lines.grep(/#{pull_request_ref}/)[0]
+      end
+    end
+
+    # @return [String] Log line of most recent merged Pull Request
+    def merged_pull_request
+      return if pull_request_ref.empty?
+
+      if most_recent_merged_pull_request
+        most_recent_merged_pull_request
+      elsif most_recent_squash_and_merged_pull_request
+        most_recent_squash_and_merged_pull_request
+      end
+    end
+
+    def check_if_any_merged_pull_request!
+      if merged_pull_request.to_s.empty?
+        if specific_pull_request_id
+          raise "Could not find the Pull Request (#{specific_pull_request_id}) inside the git history for this repo."
+        else
+          raise "No recent Pull Requests found for this repo, danger requires at least one Pull Request for the local mode.".freeze
+        end
+      end
+    end
+
+    def merged_pull_request_id
+      merged_pull_request.match(/#(?<id>[0-9]+)/)[:id]
+    end
+
+    def merged_pull_request_sha
+      merged_pull_request.split(" ".freeze).first
+    end
+  end
+end

data/lib/danger/ci_source/support/remote_finder.rb

@@ -0,0 +1,26 @@
+module Danger
+  class RemoteFinder
+    def initialize(github_host, remote_logs)
+      @github_host = github_host
+      @remote_logs = remote_logs
+    end
+
+    def call
+      remote_url_matches && remote_url_matches["repo_slug"]
+    end
+
+    private
+
+    attr_reader :remote_logs, :github_host
+
+    # @return [String] The remote URL
+    def remote
+      @remote ||= remote_logs.lines.grep(/Fetch URL/)[0].split(": ".freeze, 2)[1]
+    end
+
+    # @return [nil / MatchData] MatchData object or nil if not matched
+    def remote_url_matches
+      remote.match(%r{#{Regexp.escape github_host}(:|/)(?<repo_slug>.+/.+?)(?:\.git)?$})
+    end
+  end
+end

data/lib/danger/ci_source/surf.rb

@@ -1,4 +1,5 @@
 # http://github.com/surf-build/surf
+require "danger/request_sources/github"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/teamcity.rb

@@ -1,4 +1,6 @@
 # https://www.jetbrains.com/teamcity/
+require "danger/request_sources/github"
+require "danger/request_sources/gitlab"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/travis.rb

@@ -1,5 +1,6 @@
 # http://docs.travis-ci.com/user/osx-ci-environment/
 # http://docs.travis-ci.com/user/environment-variables/
+require "danger/request_sources/github"
 
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/xcode_server.rb

@@ -1,5 +1,6 @@
 # Following the advice from @czechboy0 https://github.com/danger/danger/issues/171
 # https://github.com/czechboy0/Buildasaur
+require "danger/request_sources/github"
 
 module Danger
   # ### CI Setup

data/lib/danger/danger_core/environment_manager.rb

@@ -1,5 +1,5 @@
 require "danger/ci_source/ci_source"
-require "danger/request_source/request_source"
+require "danger/request_sources/request_source"
 
 module Danger
   class EnvironmentManager
@@ -28,7 +28,7 @@ module Danger
         self.request_source = request_source
       end
 
-      raise "Could not find a Request Source for #{ci_klass}".red unless self.request_source
+      raise "Could not find a Request Source for #{ci_klass}\nCI: #{ci_source.inspect}".red unless self.request_source
       self.scm = self.request_source.scm
     end
 

data/lib/danger/danger_core/executor.rb

@@ -40,6 +40,7 @@ module Danger
         # 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)
 
         # Parse the local Dangerfile
@@ -52,12 +53,18 @@ module Danger
       end
     end
 
-    def post_results(dm, danger_id)
-      gh = dm.env.request_source
-      violations = dm.violation_report
-      status = dm.status_report
+    def post_results(danger_file, danger_id)
+      request_source = danger_file.env.request_source
+      violations = danger_file.violation_report
+      status = danger_file.status_report
 
-      gh.update_pull_request!(warnings: violations[:warnings], errors: violations[:errors], messages: violations[:messages], markdowns: status[:markdowns], danger_id: danger_id)
+      request_source.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_github_plugin.rb

@@ -97,7 +97,7 @@ module Danger
     # @return [String]
     #
     def pr_title
-      @github.pr_json[:title].to_s
+      @github.pr_json["title"].to_s
     end
 
     # @!group PR Metadata
@@ -105,7 +105,7 @@ module Danger
     # @return [String]
     #
     def pr_body
-      pr_json[:body].to_s
+      pr_json["body"].to_s
     end
 
     # @!group PR Metadata
@@ -113,7 +113,7 @@ module Danger
     # @return [String]
     #
     def pr_author
-      pr_json[:user][:login].to_s
+      pr_json["user"]["login"].to_s
     end
 
     # @!group PR Metadata
@@ -121,7 +121,7 @@ module Danger
     # @return [String]
     #
     def pr_labels
-      @github.issue_json[:labels].map { |l| l[:name] }
+      @github.issue_json["labels"].map { |l| l[:name] }
     end
 
     # @!group PR Commit Metadata
@@ -129,7 +129,7 @@ module Danger
     # @return [String]
     #
     def branch_for_base
-      pr_json[:base][:ref]
+      pr_json["base"]["ref"]
     end
 
     # @!group PR Commit Metadata
@@ -137,7 +137,7 @@ module Danger
     # @return [String]
     #
     def branch_for_head
-      pr_json[:head][:ref]
+      pr_json["head"]["ref"]
     end
 
     # @!group PR Commit Metadata
@@ -145,7 +145,7 @@ module Danger
     # @return [String]
     #
     def base_commit
-      pr_json[:base][:sha]
+      pr_json["base"]["sha"]
     end
 
     # @!group PR Commit Metadata
@@ -153,7 +153,7 @@ module Danger
     # @return [String]
     #
     def head_commit
-      pr_json[:head][:sha]
+      pr_json["head"]["sha"]
     end
 
     # @!group GitHub Misc
@@ -193,7 +193,7 @@ module Danger
     def html_link(paths, full_path: true)
       paths = [paths] unless paths.kind_of?(Array)
       commit = head_commit
-      repo = pr_json[:head][:repo][:html_url]
+      repo = pr_json["head"]["repo"]["html_url"]
 
       paths = paths.map do |path|
         url_path = path.start_with?("/") ? path : "/#{path}"

data/lib/danger/helpers/comment.rb

@@ -8,7 +8,7 @@ module Danger
     end
 
     def self.from_github(comment)
-      self.new(comment[:id], comment[:body])
+      self.new(comment["id"], comment["body"])
     end
 
     def self.from_gitlab(comment)

data/lib/danger/{request_source → request_sources}/github.rb

@@ -52,8 +52,8 @@ module Danger
 
       def setup_danger_branches
         # we can use a github specific feature here:
-        base_commit = self.pr_json[:base][:sha]
-        head_commit = self.pr_json[:head][:sha]
+        base_commit = self.pr_json["base"]["sha"]
+        head_commit = self.pr_json["head"]["sha"]
 
         # Next, we want to ensure that we have a version of the current branch at a known location
         self.scm.exec "branch #{EnvironmentManager.danger_base_branch} #{base_commit}"
@@ -65,7 +65,7 @@ module Danger
 
       def fetch_details
         self.pr_json = client.pull_request(ci_source.repo_slug, ci_source.pull_request_id)
-        if self.pr_json[:message] == "Moved Permanently"
+        if self.pr_json["message"] == "Moved Permanently"
           raise "Repo moved or renamed, make sure to update the git remote".red
         end
 
@@ -74,13 +74,13 @@ module Danger
       end
 
       def ignored_violations_from_pr(pr_json)
-        pr_body = pr_json[:body]
+        pr_body = pr_json["body"]
         return [] if pr_body.nil?
         pr_body.chomp.scan(/>\s*danger\s*:\s*ignore\s*"(.*)"/i).flatten
       end
 
       def fetch_issue_details(pr_json)
-        href = pr_json[:_links][:issue][:href]
+        href = pr_json["_links"]["issue"]["href"]
         self.issue_json = client.get(href)
       end
 
@@ -151,14 +151,13 @@ module Danger
         # Note: this can terminate the entire process.
         submit_pull_request_status!(warnings: warnings,
                                       errors: errors,
-                                 details_url: comment_result[:html_url])
+                                 details_url: comment_result["html_url"])
       end
 
       def submit_pull_request_status!(warnings: [], errors: [], details_url: [])
         status = (errors.count.zero? ? "success" : "failure")
         message = generate_description(warnings: warnings, errors: errors)
-
-        latest_pr_commit_ref = self.pr_json[:head][:sha]
+        latest_pr_commit_ref = self.pr_json["head"]["sha"]
 
         if latest_pr_commit_ref.empty? || latest_pr_commit_ref.nil?
           raise "Couldn't find a commit to update its status".red
@@ -176,7 +175,7 @@ module Danger
           # use a read-only GitHub account
           if errors.count > 0
             # We need to fail the actual build here
-            is_private = pr_json[:base][:repo][:private]
+            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 to set a PR status.")
             else
@@ -203,32 +202,32 @@ module Danger
 
         diff_lines = self.pr_diff.lines
         pr_comments = client.pull_request_comments(ci_source.repo_slug, ci_source.pull_request_id)
-        danger_comments = pr_comments.select { |comment| comment[:body].include?("generated_by_#{danger_id}") }
+        danger_comments = pr_comments.select { |comment| comment["body"].include?("generated_by_#{danger_id}") }
         non_danger_comments = pr_comments - danger_comments
 
-        submit_inline_comments_for_kind!("warning", warnings, diff_lines, danger_comments, previous_violations[:warning], danger_id: danger_id)
-        submit_inline_comments_for_kind!("no_entry_sign", errors, diff_lines, danger_comments, previous_violations[:error], danger_id: danger_id)
-        submit_inline_comments_for_kind!("book", messages, diff_lines, danger_comments, previous_violations[:message], danger_id: danger_id)
+        submit_inline_comments_for_kind!("warning", warnings, diff_lines, danger_comments, previous_violations["warning"], danger_id: danger_id)
+        submit_inline_comments_for_kind!("no_entry_sign", errors, diff_lines, danger_comments, previous_violations["error"], danger_id: danger_id)
+        submit_inline_comments_for_kind!("book", messages, diff_lines, danger_comments, previous_violations["message"], danger_id: danger_id)
         submit_inline_comments_for_kind!(nil, markdowns, diff_lines, danger_comments, [], danger_id: danger_id)
 
         # submit removes from the array all comments that are still in force
         # so we strike out all remaining ones
         danger_comments.each do |comment|
-          violation = violations_from_table(comment[:body]).first
+          violation = violations_from_table(comment["body"]).first
           if !violation.nil? && violation.sticky
             body = generate_inline_comment_body("white_check_mark", violation, danger_id: danger_id, resolved: true, template: "github")
-            client.update_pull_request_comment(ci_source.repo_slug, comment[:id], body)
+            client.update_pull_request_comment(ci_source.repo_slug, comment["id"], body)
           else
             # We remove non-sticky violations that have no replies
             # Since there's no direct concept of a reply in GH, we simply consider
             # the existance of non-danger comments in that line as replies
             replies = non_danger_comments.select do |potential|
-              potential[:path] == comment[:path] &&
-                potential[:position] == comment[:position] &&
-                potential[:commit_id] == comment[:commit_id]
+              potential["path"] == comment["path"] &&
+                potential["position"] == comment["position"] &&
+                potential["commit_id"] == comment["commit_id"]
             end
 
-            client.delete_pull_request_comment(ci_source.repo_slug, comment[:id]) if replies.empty?
+            client.delete_pull_request_comment(ci_source.repo_slug, comment["id"]) if replies.empty?
           end
         end
       end
@@ -240,7 +239,7 @@ module Danger
       end
 
       def submit_inline_comments_for_kind!(emoji, messages, diff_lines, danger_comments, previous_violations, danger_id: "danger")
-        head_ref = pr_json[:head][:sha]
+        head_ref = pr_json["head"]["sha"]
         previous_violations ||= []
         is_markdown_content = emoji.nil?
 
@@ -265,13 +264,13 @@ module Danger
           end
 
           matching_comments = danger_comments.select do |comment_data|
-            if comment_data[:path] == m.file && comment_data[:commit_id] == head_ref && comment_data[:position] == position
+            if comment_data["path"] == m.file && comment_data["commit_id"] == head_ref && comment_data["position"] == position
               # Parse it to avoid problems with strikethrough
-              violation = violations_from_table(comment_data[:body]).first
+              violation = violations_from_table(comment_data["body"]).first
               if violation
                 messages_are_equivalent(violation, m)
               else
-                comment_data[:body] == body
+                comment_data["body"] == body
               end
             else
               false
@@ -287,7 +286,7 @@ module Danger
 
             # Update the comment to remove the strikethrough if present
             comment = matching_comments.first
-            client.update_pull_request_comment(ci_source.repo_slug, comment[:id], body)
+            client.update_pull_request_comment(ci_source.repo_slug, comment["id"], body)
           end
 
           # Remove this element from the array
@@ -360,7 +359,7 @@ module Danger
       end
 
       def markdown_link_to_message(message, hide_link)
-        url = "https://github.com/#{ci_source.repo_slug}/blob/#{pr_json[:head][:sha]}/#{message.file}#L#{message.line}"
+        url = "https://github.com/#{ci_source.repo_slug}/blob/#{pr_json['head']['sha']}/#{message.file}#L#{message.line}"
 
         if hide_link
           "<span data-href=\"#{url}\"/>"
@@ -371,7 +370,7 @@ module Danger
 
       # @return [String] The organisation name, is nil if it can't be detected
       def organisation
-        matched = self.issue_json[:repository_url].match(%r{repos\/(.*)\/})
+        matched = self.issue_json["repository_url"].match(%r{repos\/(.*)\/})
         return matched[1] if matched && matched[1]
       rescue
         nil

data/lib/danger/version.rb

@@ -1,4 +1,4 @@
 module Danger
-  VERSION = "3.3.0".freeze
+  VERSION = "3.3.1".freeze
   DESCRIPTION = "Like Unit Tests, but for your Team Culture.".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger
 version: !ruby/object:Gem::Version
-  version: 3.3.0
+  version: 3.3.1
 platform: ruby
 authors:
 - Orta Therox
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-16 00:00:00.000000000 Z
+date: 2016-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide
@@ -249,6 +249,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  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: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -372,6 +386,8 @@ files:
 - lib/danger/ci_source/jenkins.rb
 - lib/danger/ci_source/local_git_repo.rb
 - lib/danger/ci_source/semaphore.rb
+- lib/danger/ci_source/support/merged_pull_request_finder.rb
+- lib/danger/ci_source/support/remote_finder.rb
 - lib/danger/ci_source/surf.rb
 - lib/danger/ci_source/teamcity.rb
 - lib/danger/ci_source/travis.rb
@@ -414,13 +430,13 @@ files:
 - lib/danger/plugin_support/plugin_linter.rb
 - lib/danger/plugin_support/plugin_parser.rb
 - lib/danger/plugin_support/templates/readme_table.html.erb
-- lib/danger/request_source/bitbucket_cloud.rb
-- lib/danger/request_source/bitbucket_cloud_api.rb
-- lib/danger/request_source/bitbucket_server.rb
-- lib/danger/request_source/bitbucket_server_api.rb
-- lib/danger/request_source/github.rb
-- lib/danger/request_source/gitlab.rb
-- lib/danger/request_source/request_source.rb
+- lib/danger/request_sources/bitbucket_cloud.rb
+- lib/danger/request_sources/bitbucket_cloud_api.rb
+- lib/danger/request_sources/bitbucket_server.rb
+- lib/danger/request_sources/bitbucket_server_api.rb
+- lib/danger/request_sources/github.rb
+- lib/danger/request_sources/gitlab.rb
+- lib/danger/request_sources/request_source.rb
 - lib/danger/scm_source/git_repo.rb
 - lib/danger/version.rb
 homepage: https://github.com/danger/danger