danger 9.5.3 → 9.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1a29ac64cb5013e28a0e137ba85672f035a3116508c15815ff34e2e4c2efe71
-  data.tar.gz: ccd8f3bf454de07224648c91fcff6f4e63d8173b50215d769de44230b4621e16
+  metadata.gz: 3c20de89d82a1a32f926300a84b0a1313a7d3c1d5c35c91b4eed2667eec5788c
+  data.tar.gz: 208508f00a6542efe5c5f7357a87f943e50384b7c999771e669c9c585df4e3a7
 SHA512:
-  metadata.gz: 11a8ad39bcfbddb55c4683e7a7852d79ac17f74f5da55dcfabc796ac4b83a5f4b45d2ad5de53cc8bfdb47034c0572d546ff225b021c4420a751c3d9e0cd9962e
-  data.tar.gz: 6dd015355fb17c8bf4a4f79eec81e4e755b1e6eb94268d55809f4146b41367ed474969ce08358e8ef2deb05998af89ec04414ae0caed6407ff28ef24c374452d
+  metadata.gz: bc94ca5d7d28214e3cf4166dce28369a1a22e7173eee2d5c1c0a34a826c32c97c7d58e37ba66a9e87a2ea3540ffe6674ea64be705e0ca920166df856b7bcd116
+  data.tar.gz: 3929c665775cf4af212c36a9ea898b9850dc3358d79597707fdd4b166d4b605777d85c793b54eeb72ac8e3207be217b17be602a3d395c2a749962d076bc5aaf1

data/bin/danger

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
+
 $LOAD_PATH.push File.expand_path("../lib", __dir__)
 
 require "danger"

data/lib/danger/ci_source/appcenter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.microsoft.com/en-us/appcenter/build/custom/variables/
 require "uri"
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/appcircle.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.appcircle.io/environment-variables/managing-variables
 # https://docs.appcircle.io/build/build-profile-configuration#environment-variables-configuration
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/appveyor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://www.appveyor.com/docs/build-configuration/
 module Danger
   # ### CI Setup

data/lib/danger/ci_source/azure_pipelines.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.microsoft.com/en-us/azure/devops/pipelines/build/variables
 require "uri"
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/bamboo.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "set"
 
 module Danger

data/lib/danger/ci_source/bitbucket_pipelines.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   # ### CI Setup
   #

data/lib/danger/ci_source/bitrise.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://devcenter.bitrise.io/docs/available-environment-variables
 require "danger/request_sources/github/github"
 require "danger/request_sources/gitlab"
@@ -49,7 +51,6 @@ module Danger
       self.pull_request_id = env["BITRISE_PULL_REQUEST"]
       self.repo_url = env["GIT_REPOSITORY_URL"]
 
-      matcher_url = self.repo_url
       self.repo_slug = repo_slug_from(self.repo_url)
     end
 

data/lib/danger/ci_source/buddybuild.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   # ### CI Setup
   #

data/lib/danger/ci_source/buildkite.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://buildkite.com/docs/agent/osx
 # https://buildkite.com/docs/guides/environment-variables
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/ci_source.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "set"
 
 module Danger

data/lib/danger/ci_source/circle.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://circleci.com/docs/environment-variables
 require "uri"
 require "danger/ci_source/circle_api"
@@ -78,7 +80,7 @@ module Danger
       pr_path = URI.parse(pr_url).path.split("/")
       if pr_path.count == 5
         # The first one is an extra slash, ignore it
-        self.repo_slug = pr_path[1] + "/" + pr_path[2]
+        self.repo_slug = "#{pr_path[1]}/#{pr_path[2]}"
         self.pull_request_id = pr_path[4]
 
       else

data/lib/danger/ci_source/circle_api.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "faraday"
 
 module Danger
@@ -15,10 +17,10 @@ module Danger
       url = env["CI_PULL_REQUEST"]
 
       if url.nil? && !env["CIRCLE_PROJECT_USERNAME"].nil? && !env["CIRCLE_PROJECT_REPONAME"].nil?
-        repo_slug = env["CIRCLE_PROJECT_USERNAME"] + "/" + env["CIRCLE_PROJECT_REPONAME"]
+        repo_slug = "#{env['CIRCLE_PROJECT_USERNAME']}/#{env['CIRCLE_PROJECT_REPONAME']}"
         if !env["CIRCLE_PR_NUMBER"].nil?
           host = env["DANGER_GITHUB_HOST"] || "github.com"
-          url = "https://" + host + "/" + repo_slug + "/pull/" + env["CIRCLE_PR_NUMBER"]
+          url = "https://#{host}/#{repo_slug}/pull/#{env['CIRCLE_PR_NUMBER']}"
         else
           token = env["DANGER_CIRCLE_CI_API_TOKEN"]
           url = fetch_pull_request_url(repo_slug, env["CIRCLE_BUILD_NUM"], token)
@@ -35,7 +37,7 @@ module Danger
     def fetch_pull_request_url(repo_slug, build_number, token)
       build_json = fetch_build(repo_slug, build_number, token)
       pull_requests = build_json[:pull_requests]
-      return nil unless pull_requests && pull_requests.first
+      return nil unless pull_requests&.first
 
       pull_requests.first[:url]
     end

data/lib/danger/ci_source/cirrus.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/request_sources/github/github"
 
 module Danger

data/lib/danger/ci_source/code_build.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.aws.amazon.com/codebuild/latest/userguide/build-env-ref-env-vars.html
 require "danger/request_sources/github/github"
 

data/lib/danger/ci_source/codefresh.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://semaphoreci.com/docs/available-environment-variables.html
 require "danger/request_sources/github/github"
 
@@ -35,7 +37,7 @@ module Danger
       return "" if env["CF_REPO_OWNER"].to_s.empty?
       return "" if env["CF_REPO_NAME"].to_s.empty?
 
-      "#{env['CF_REPO_OWNER']}/#{env['CF_REPO_NAME']}".downcase!
+      "#{env['CF_REPO_OWNER']}/#{env['CF_REPO_NAME']}".downcase
     end
 
     def initialize(env)

data/lib/danger/ci_source/codemagic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.codemagic.io/building/environment-variables/
 
 module Danger

data/lib/danger/ci_source/codeship.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://semaphoreci.com/docs/available-environment-variables.html
 require "danger/request_sources/github/github"
 

data/lib/danger/ci_source/concourse.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "git"
 require "danger/request_sources/local_only"
 

data/lib/danger/ci_source/dotci.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/request_sources/github/github"
 
 module Danger

data/lib/danger/ci_source/drone.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://readme.drone.io/usage/variables/
 require "danger/request_sources/github/github"
 require "danger/request_sources/gitlab"

data/lib/danger/ci_source/github_actions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/request_sources/github/github"
 
 module Danger

data/lib/danger/ci_source/gitlab_ci.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://docs.gitlab.com/ce/ci/variables/README.html
 require "uri"
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/jenkins.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # 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/github"

data/lib/danger/ci_source/local_git_repo.rb

@@ -38,7 +38,8 @@ module Danger
         Danger::RequestSources::GitHub,
         Danger::RequestSources::BitbucketServer,
         Danger::RequestSources::BitbucketCloud,
-        Danger::RequestSources::VSTS
+        Danger::RequestSources::VSTS,
+        Danger::RequestSources::GitLab
       ]
     end
 
@@ -74,7 +75,8 @@ module Danger
 
     def find_remote_info(env)
       if given_pull_request_url?(env)
-        FindRepoInfoFromURL.new(env["LOCAL_GIT_PR_URL"]).call
+        pr_url = env["LOCAL_GIT_PR_URL"] || env["LOCAL_GIT_MR_URL"]
+        FindRepoInfoFromURL.new(pr_url).call
       else
         FindRepoInfoFromLogs.new(
           env["DANGER_GITHUB_HOST"] || "github.com",
@@ -85,15 +87,16 @@ module Danger
 
     def find_pull_request(env)
       if given_pull_request_url?(env)
+        remote_url = env["LOCAL_GIT_PR_URL"] || env["LOCAL_GIT_MR_URL"]
         PullRequestFinder.new(
           remote_info.id,
           remote_info.slug,
           remote: true,
-          remote_url: env["LOCAL_GIT_PR_URL"]
+          remote_url: remote_url
         ).call(env: env)
       else
         PullRequestFinder.new(
-          env.fetch("LOCAL_GIT_PR_ID") { "" },
+          env.fetch("LOCAL_GIT_PR_ID") { env.fetch("LOCAL_GIT_MR_ID", "") },
           remote_info.slug,
           remote: false,
           git_logs: run_git("log --oneline -1000000")
@@ -102,7 +105,8 @@ module Danger
     end
 
     def given_pull_request_url?(env)
-      env["LOCAL_GIT_PR_URL"] && !env["LOCAL_GIT_PR_URL"].empty?
+      (env["LOCAL_GIT_PR_URL"] && !env["LOCAL_GIT_PR_URL"].empty?) ||
+        (env["LOCAL_GIT_MR_URL"] && !env["LOCAL_GIT_MR_URL"].empty?)
     end
 
     def sha

data/lib/danger/ci_source/screwdriver.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://screwdriver.cd
 # https://docs.screwdriver.cd/user-guide/environment-variables
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/semaphore.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://docs.semaphoreci.com/article/12-environment-variables
 require "danger/request_sources/github/github"
 

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

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Danger
   class Commits
     def initialize(base_head)
-      @base_head = base_head.strip.split(" ".freeze)
+      @base_head = base_head.strip.split(" ")
     end
 
     def base

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/ci_source/support/repo_info"
 
 module Danger
@@ -20,7 +22,7 @@ module Danger
     attr_reader :remote_logs, :github_host
 
     def remote
-      remote_logs.lines.grep(/Fetch URL/)[0].split(": ".freeze, 2)[1]
+      remote_logs.lines.grep(/Fetch URL/)[0].split(": ", 2)[1]
     end
 
     def regexp

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/ci_source/support/repo_info"
 
 module Danger
@@ -31,7 +33,9 @@ module Danger
       else
         matched = url.match(REGEXP)
         if matched
-          RepoInfo.new(matched[:slug], matched[:id])
+          # Clean up the slug to remove any trailing dashes or slashes that might be part of the GitLab URL format
+          clean_slug = matched[:slug].gsub(%r{[-/]+$}, "")
+          RepoInfo.new(clean_slug, matched[:id])
         end
       end
     end

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

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Danger
   class LocalPullRequest
     attr_reader :pull_request_id, :sha
 
     def initialize(log_line)
       @pull_request_id = log_line.match(/#(?<id>[0-9]+)/)[:id]
-      @sha = log_line.split(" ".freeze).first
+      @sha = log_line.split(" ").first
     end
 
     def valid?

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   class NoPullRequest
     def valid?

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   class NoRepoInfo
     attr_reader :slug, :id

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

@@ -81,6 +81,12 @@ module Danger
           remote_pull_request.head.sha,
           remote_pull_request.base.sha
         )
+      when :gitlab
+        RemotePullRequest.new(
+          remote_pull_request.iid.to_s,
+          remote_pull_request.diff_refs.head_sha,
+          remote_pull_request.diff_refs.base_sha
+        )
       when :vsts
         RemotePullRequest.new(
           remote_pull_request[:pullRequestId].to_s,
@@ -93,7 +99,13 @@ module Danger
     end
 
     def find_remote_pull_request(env)
-      client(env).pull_request(repo_slug, specific_pull_request_id)
+      scm_provider = find_scm_provider(remote_url)
+
+      if scm_provider == :gitlab
+        client(env).merge_request(repo_slug, specific_pull_request_id)
+      else
+        client(env).pull_request(repo_slug, specific_pull_request_id)
+      end
     end
 
     def both_present?
@@ -138,50 +150,77 @@ module Danger
 
       case scm_provider
       when :bitbucket_cloud
-        require "danger/request_sources/bitbucket_cloud_api"
-        branch_name = ENV["DANGER_BITBUCKET_TARGET_BRANCH"] # Optional env variable (specifying the target branch) to help find the PR.
-        RequestSources::BitbucketCloudAPI.new(repo_slug, specific_pull_request_id, branch_name, env)
-
+        bitbucket_cloud_client(env)
       when :bitbucket_server
-        require "danger/request_sources/bitbucket_server_api"
-        project, slug = repo_slug.split("/")
-        RequestSources::BitbucketServerAPI.new(project, slug, specific_pull_request_id, env)
-
+        bitbucket_server_client(env)
       when :vsts
-        require "danger/request_sources/vsts_api"
-        RequestSources::VSTSAPI.new(repo_slug, specific_pull_request_id, env)
-
+        vsts_client(env)
+      when :gitlab
+        gitlab_client(env)
       when :github
-        require "octokit"
-        access_token = ENV["DANGER_GITHUB_API_TOKEN"]
-        bearer_token = ENV["DANGER_GITHUB_BEARER_TOKEN"]
-        if bearer_token && !bearer_token.empty?
-          Octokit::Client.new(bearer_token: bearer_token, api_endpoint: api_url)
-        elsif access_token && !access_token.empty?
-          Octokit::Client.new(access_token: access_token, api_endpoint: api_url)
-        else
-          raise "No API token given, please provide one using `DANGER_GITHUB_API_TOKEN` or `DANGER_GITHUB_BEARER_TOKEN`"
-        end
+        github_client(env)
       else
         raise "SCM provider not supported: #{scm_provider}"
       end
     end
 
+    def bitbucket_cloud_client(env)
+      require "danger/request_sources/bitbucket_cloud_api"
+      branch_name = ENV["DANGER_BITBUCKET_TARGET_BRANCH"] # Optional env variable (specifying the target branch) to help find the PR.
+      RequestSources::BitbucketCloudAPI.new(repo_slug, specific_pull_request_id, branch_name, env)
+    end
+
+    def bitbucket_server_client(env)
+      require "danger/request_sources/bitbucket_server_api"
+      project, slug = repo_slug.split("/")
+      RequestSources::BitbucketServerAPI.new(project, slug, specific_pull_request_id, env)
+    end
+
+    def vsts_client(env)
+      require "danger/request_sources/vsts_api"
+      RequestSources::VSTSAPI.new(repo_slug, specific_pull_request_id, env)
+    end
+
+    def gitlab_client(env)
+      require "gitlab"
+      token = env&.fetch("DANGER_GITLAB_API_TOKEN", nil) || ENV["DANGER_GITLAB_API_TOKEN"]
+      if token && !token.empty?
+        endpoint = env&.fetch("DANGER_GITLAB_API_BASE_URL", nil) || env&.fetch("CI_API_V4_URL", nil) || ENV["DANGER_GITLAB_API_BASE_URL"] || ENV.fetch("CI_API_V4_URL", "https://gitlab.com/api/v4")
+        Gitlab.client(endpoint: endpoint, private_token: token)
+      else
+        raise "No API token given, please provide one using `DANGER_GITLAB_API_TOKEN`"
+      end
+    end
+
+    def github_client(env)
+      require "octokit"
+      access_token = env&.fetch("DANGER_GITHUB_API_TOKEN", nil) || ENV["DANGER_GITHUB_API_TOKEN"]
+      bearer_token = env&.fetch("DANGER_GITHUB_BEARER_TOKEN", nil) || ENV["DANGER_GITHUB_BEARER_TOKEN"]
+      if bearer_token && !bearer_token.empty?
+        Octokit::Client.new(bearer_token: bearer_token, api_endpoint: api_url)
+      elsif access_token && !access_token.empty?
+        Octokit::Client.new(access_token: access_token, api_endpoint: api_url)
+      else
+        raise "No API token given, please provide one using `DANGER_GITHUB_API_TOKEN` or `DANGER_GITHUB_BEARER_TOKEN`"
+      end
+    end
+
     def api_url
       ENV.fetch("DANGER_GITHUB_API_HOST") do
-        ENV.fetch("DANGER_GITHUB_API_BASE_URL") do
-          "https://api.github.com/"
-        end
+        ENV.fetch("DANGER_GITHUB_API_BASE_URL", "https://api.github.com/")
       end
     end
 
     def find_scm_provider(remote_url)
-      if remote_url =~ %r{/bitbucket.org/}
+      case remote_url
+      when %r{/bitbucket.org/}
         :bitbucket_cloud
-      elsif remote_url =~ %r{/pull-requests/}
+      when %r{/pull-requests/}
         :bitbucket_server
-      elsif remote_url =~ /\.visualstudio\.com/i || remote_url =~ /dev\.azure\.com/i
+      when /\.visualstudio\.com/i, /dev\.azure\.com/i
         :vsts
+      when /gitlab\.com/, %r{-/merge_requests/}
+        :gitlab
       else
         :github
       end

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   class RemotePullRequest
     attr_reader :pull_request_id, :sha, :head, :base

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

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   class RepoInfo
     attr_reader :slug, :id

data/lib/danger/ci_source/surf.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://github.com/surf-build/surf
 require "danger/request_sources/github/github"
 

data/lib/danger/ci_source/teamcity.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # https://www.jetbrains.com/teamcity/
 require "danger/request_sources/github/github"
 require "danger/request_sources/gitlab"
@@ -65,6 +67,14 @@ module Danger
   # export BITBUCKET_BRANCH_NAME="%teamcity.build.branch%"
   # ```
   #
+  # Optionally, you can set `BITBUCKET_PULL_REQUEST_ID` to skip the API call that resolves
+  # the pull request ID from the branch name. If you are using the TeamCity
+  # [Pull Requests](https://www.jetbrains.com/help/teamcity/pull-requests.html) build feature:
+  #
+  # ```sh
+  # export BITBUCKET_PULL_REQUEST_ID="%teamcity.pullRequest.number%"
+  # ```
+  #
   # #### BitBucket Server
   #
   # You will need to add the following environment variables as build parameters or by exporting them inside your
@@ -149,8 +159,11 @@ module Danger
       self.repo_url        = env["BITBUCKETSERVER_REPO_URL"]
     end
 
-    # This is a little hacky, because Bitbucket doesn't provide us a PR id
+    # Uses BITBUCKET_PULL_REQUEST_ID from the environment if available,
+    # otherwise falls back to fetching the PR id from the Bitbucket API.
     def bitbucket_pr_from_env(env)
+      return env["BITBUCKET_PULL_REQUEST_ID"].to_i if env["BITBUCKET_PULL_REQUEST_ID"]
+
       branch_name = env["BITBUCKET_BRANCH_NAME"]
       repo_slug   = env["BITBUCKET_REPO_SLUG"]
       begin

data/lib/danger/ci_source/travis.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # http://docs.travis-ci.com/user/osx-ci-environment/
 # http://docs.travis-ci.com/user/environment-variables/
 require "danger/request_sources/github/github"

data/lib/danger/ci_source/xcode_cloud.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Danger
   # ### CI Setup
   #

data/lib/danger/ci_source/xcode_server.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Following the advice from @czechboy0 https://github.com/danger/danger/issues/171
 # https://github.com/czechboy0/Buildasaur
 require "danger/request_sources/github/github"

data/lib/danger/clients/rubygems_client.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Danger
   class RubyGemsClient
-    API_URL = "https://rubygems.org/api/v1/versions/danger/latest.json".freeze
-    DUMMY_VERSION = "0.0.0".freeze
+    API_URL = "https://rubygems.org/api/v1/versions/danger/latest.json"
+    DUMMY_VERSION = "0.0.0"
 
     def self.latest_danger_version
       require "json"

data/lib/danger/commands/dangerfile/gem.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "claide_plugin"
 require "danger/commands/dangerfile/init"
 
@@ -17,7 +19,7 @@ module Danger
 
     def initialize(argv)
       @name = argv.shift_argument
-      prefix = "dangerfile" + "-"
+      prefix = "dangerfile-"
       unless @name.nil? || @name.empty? || @name.start_with?(prefix)
         @name = prefix + @name.dup
       end

data/lib/danger/commands/dangerfile/init.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/danger_core/dangerfile_generator"
 
 # Mainly so we can have a nice structure for commands
@@ -24,7 +26,7 @@ module Danger
     def run
       content = DangerfileGenerator.create_dangerfile(".", cork)
       File.write("Dangerfile", content)
-      cork.puts "Created" + "./Dangerfile".green
+      cork.puts "Created#{'./Dangerfile'.green}"
     end
   end
 end

data/lib/danger/commands/dry_run.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/commands/local_helpers/pry_setup"
 require "fileutils"