danger 9.5.2 → 9.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 17279e83964586e1eef46c40740b9c0fb839ea85a3c3d367a898c9b664767eca
-  data.tar.gz: 6afc0eb2cb97f22d988bf56cb0fe48063aa5eab060471cff2d95c0bc8435b3da
+  metadata.gz: 3c20de89d82a1a32f926300a84b0a1313a7d3c1d5c35c91b4eed2667eec5788c
+  data.tar.gz: 208508f00a6542efe5c5f7357a87f943e50384b7c999771e669c9c585df4e3a7
 SHA512:
-  metadata.gz: ee7544abd31b5bd6320efdc79668169d0c5bfc0aa873b2d971bdfb043ab08bd3e05149ddc714cbcd98257c7bd0c815e4899afd573c33ff5d815626fdf01c1c01
-  data.tar.gz: fbe7c247fee1a41cb6e212141896f7282547f91f8ad0b59f36ae18bab2e32783e1b3ff22db1231e30e6bb5d3feaa98aea67099f114eaa9a45e30dc2478d562b0
+  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

@@ -96,7 +96,7 @@ module Danger
         ).call(env: env)
       else
         PullRequestFinder.new(
-          env.fetch("LOCAL_GIT_PR_ID") { env.fetch("LOCAL_GIT_MR_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")

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

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

@@ -207,20 +207,19 @@ module Danger
 
     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
-      elsif remote_url =~ /gitlab\.com/ || remote_url =~ %r{-/merge_requests/}
+      when /gitlab\.com/, %r{-/merge_requests/}
         :gitlab
       else
         :github

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"
 

data/lib/danger/commands/init.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "danger/commands/init_helpers/interviewer"
 require "danger/danger_core/dangerfile_generator"
 require "danger/ci_source/local_git_repo"
@@ -18,7 +20,7 @@ module Danger
     end
 
     def initialize(argv)
-      @bot_name = File.basename(Dir.getwd).split(".").first.capitalize + "Bot"
+      @bot_name = "#{File.basename(Dir.getwd).split('.').first.capitalize}Bot"
       super
       @ui = Interviewer.new(cork)
       ui.no_delay = argv.flag?("impatient", false)
@@ -63,7 +65,7 @@ module Danger
 
       ui.say "cat #{Dir.pwd}/Dangerfile\n".blue
       content.lines.each do |l|
-        ui.say "  " + l.chomp.green
+        ui.say "  #{l.chomp.green}"
       end
       ui.say ""
       ui.pause 2
@@ -118,7 +120,7 @@ module Danger
 
       if considered_an_oss_repo?
         ui.say "For Open Source projects, I'd recommend giving the token the smallest scope possible."
-        ui.say "This means only providing access to " + "public_repo".yellow + " in the token.\n\n"
+        ui.say "This means only providing access to #{'public_repo'.yellow} in the token.\n\n"
         ui.pause 1
         ui.say "This token limits Danger's abilities to just writing comments on OSS projects. I recommend"
         ui.say "this because the token can quite easily be extracted from the environment via pull requests."
@@ -126,10 +128,10 @@ module Danger
         ui.say "\nIt is important that you do not store this token in your repository, as GitHub will automatically revoke it when pushed.\n"
       elsif @is_open_source == "closed"
         ui.say "For Closed Source projects, I'd recommend giving the token access to the whole repo scope."
-        ui.say "This means only providing access to " + "repo".yellow + ", and its children in the token.\n\n"
+        ui.say "This means only providing access to #{'repo'.yellow}, and its children in the token.\n\n"
         ui.pause 1
-        ui.say "It's worth noting that you " + "should not".bold.white + " re-use this token for OSS repos."
-        ui.say "Make a new one for those repos with just " + "public_repo".yellow + "."
+        ui.say "It's worth noting that you #{'should not'.bold.white} re-use this token for OSS repos."
+        ui.say "Make a new one for those repos with just #{'public_repo'.yellow}."
         ui.pause 1
         ui.say "Additionally, don't forget to add your new GitHub account as a collaborator to your Closed Source project."
       end
@@ -184,13 +186,13 @@ module Danger
       danger = "bundle exec danger".yellow
       config = YAML.load(File.read(".travis.yml"))
       if config.kind_of?(Hash) && config["script"]
-        ui.say "Add " + "- ".yellow + danger + " as a new step in the " + "script".yellow + " section of your .travis.yml file."
+        ui.say "Add #{'- '.yellow}#{danger} as a new step in the #{'script'.yellow} section of your .travis.yml file."
       else
-        ui.say "I'd recommend adding " + "before_script: ".yellow + danger + " to the script section of your .travis.yml file."
+        ui.say "I'd recommend adding #{'before_script: '.yellow}#{danger} to the script section of your .travis.yml file."
       end
 
       ui.pause 1
-      ui.say "You shouldn't use " + "after_success, after_failure, after_script".red + " as they cannot fail your builds."
+      ui.say "You shouldn't use #{'after_success, after_failure, after_script'.red} as they cannot fail your builds."
     end
 
     def uses_circle
@@ -199,9 +201,9 @@ module Danger
 
       if config.kind_of?(Hash) && config["test"]
         if config["test"]["post"]
-          ui.say "Add " + danger + " as a new step in the " + "test:post:".yellow + " section of your circle.yml file."
+          ui.say "Add #{danger} as a new step in the #{'test:post:'.yellow} section of your circle.yml file."
         else
-          ui.say "Add " + danger + " as a new step in the " + "test:override:".yellow + " section of your circle.yml file."
+          ui.say "Add #{danger} as a new step in the #{'test:override:'.yellow} section of your circle.yml file."
         end
       else
         ui.say "Add this to the bottom of your circle.yml file:"
@@ -214,7 +216,7 @@ module Danger
     def unsure_ci
       danger = "bundle exec danger".yellow
       ui.say "As I'm not sure what CI you want to run Danger on based on the files in your repo, I'll just offer some generic"
-      ui.say "advice. You want to run " + danger + " after your tests have finished running, it should still be during the testing"
+      ui.say "advice. You want to run #{danger} after your tests have finished running, it should still be during the testing"
       ui.say "process so the build can fail."
     end
 
@@ -222,7 +224,7 @@ module Danger
       # https://travis-ci.org/artsy/eigen/settings
       ui.say "In order to add an environment variable, go to:"
       ui.link "https://travis-ci.org/#{current_repo_slug}/settings"
-      ui.say "\nThe name is " + "DANGER_GITHUB_API_TOKEN".yellow + " and the value is the GitHub Personal Access Token."
+      ui.say "\nThe name is #{'DANGER_GITHUB_API_TOKEN'.yellow} and the value is the GitHub Personal Access Token."
       if @is_open_source
         ui.say 'Make sure to have "Display value in build log" enabled.'
       end
@@ -239,7 +241,7 @@ module Danger
         ui.say "I'll give you a minute to read it..."
         ui.wait_for_return
 
-        ui.say "On danger/danger we turn on " + "Permissive building of fork pull requests".yellow + " this exposes the token to Danger"
+        ui.say "On danger/danger we turn on #{'Permissive building of fork pull requests'.yellow} this exposes the token to Danger"
         ui.say "You can find this setting at:"
         ui.link "https://circleci.com/gh/#{current_repo_slug}/edit#advanced-settings\n"
         ui.say "I'll hold..."
@@ -248,12 +250,12 @@ module Danger
 
       ui.say "In order to expose an environment variable, go to:"
       ui.link "https://circleci.com/gh/#{current_repo_slug}/edit#env-vars"
-      ui.say "The name is " + "DANGER_GITHUB_API_TOKEN".yellow + " and the value is the GitHub Personal Access Token."
+      ui.say "The name is #{'DANGER_GITHUB_API_TOKEN'.yellow} and the value is the GitHub Personal Access Token."
     end
 
     def unsure_token
-      ui.say "You need to expose a token called " + "DANGER_GITHUB_API_TOKEN".yellow + " and the value is the GitHub Personal Access Token."
-      ui.say "Depending on the CI system, this may need to be done on the machine (in the " + "~/.bashprofile".yellow + ") or in a web UI somewhere."
+      ui.say "You need to expose a token called #{'DANGER_GITHUB_API_TOKEN'.yellow} and the value is the GitHub Personal Access Token."
+      ui.say "Depending on the CI system, this may need to be done on the machine (in the #{'~/.bashprofile'.yellow}) or in a web UI somewhere."
       ui.say "We have a guide for all supported CI systems on danger.systems:"
       ui.link "https://danger.systems/guides/getting_started.html#setting-up-danger-to-run-on-your-ci"
     end
@@ -270,9 +272,9 @@ module Danger
 
     def info
       ui.header "Useful info"
-      ui.say "- One of the best ways to test out new rules locally is via " + "bundle exec danger pr".yellow + "."
+      ui.say "- One of the best ways to test out new rules locally is via #{'bundle exec danger pr'.yellow}."
       ui.pause 0.6
-      ui.say "- You can have Danger output all of her variables to the console via the " + "--verbose".yellow + " option."
+      ui.say "- You can have Danger output all of her variables to the console via the #{'--verbose'.yellow} option."
       ui.pause 0.6
       ui.say "- You can look at the following Dangerfiles to get some more ideas:"
       ui.pause 0.6
@@ -286,7 +288,7 @@ module Danger
       ui.pause 0.6
 
       ui.say "And you're good to go. Danger is a collaboration between Orta Therox, Gem 'Danger' McShane and Felix Krause."
-      ui.say "If you like Danger, let others know. If you want to know more, follow " + "@orta".yellow + " and " + "@KrauseFx".yellow + " on Twitter."
+      ui.say "If you like Danger, let others know. If you want to know more, follow #{'@orta'.yellow} and #{'@KrauseFx'.yellow} on Twitter."
       ui.say "If you don't like Danger, help us improve the project! xxx"
     end