lib/danger/ci_source/circle.rb in danger-0.8.5 vs lib/danger/ci_source/circle.rb in danger-0.9.0

- old
+ new

@@ -1,16 +1,16 @@ # https://circleci.com/docs/environment-variables -require 'uri' -require 'danger/ci_source/circle_api' +require "uri" +require "danger/ci_source/circle_api" module Danger module CISource class CircleCI < CI def self.validates?(env) - return false unless env['CIRCLE_BUILD_NUM'] - return false unless env['CIRCLE_PROJECT_USERNAME'] - return false unless env['CIRCLE_PROJECT_REPONAME'] + return false unless env["CIRCLE_BUILD_NUM"] + return false unless env["CIRCLE_PROJECT_USERNAME"] + return false unless env["CIRCLE_PROJECT_REPONAME"] return true end def supported_request_sources @@ -25,29 +25,29 @@ build_json = client.fetch_build(repo_slug, build_number) build_json[:pull_request_urls].first end def pull_request_url(env) - url = env['CI_PULL_REQUEST'] + 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'] - url = fetch_pull_request_url(repo_slug, env['CIRCLE_BUILD_NUM']) + if url.nil? && !env["CIRCLE_PROJECT_USERNAME"].nil? && !env["CIRCLE_PROJECT_REPONAME"].nil? + repo_slug = env["CIRCLE_PROJECT_USERNAME"] + "/" + env["CIRCLE_PROJECT_REPONAME"] + url = fetch_pull_request_url(repo_slug, env["CIRCLE_BUILD_NUM"]) end url end def initialize(env) self.repo_url = GitRepo.new.origins # CircleCI doesn't provide a repo url env variable :/ - @circle_token = env['CIRCLE_CI_API_TOKEN'] + @circle_token = env["CIRCLE_CI_API_TOKEN"] url = pull_request_url(env) - if URI.parse(url).path.split('/').count == 5 - paths = URI.parse(url).path.split('/') + if URI.parse(url).path.split("/").count == 5 + paths = URI.parse(url).path.split("/") # The first one is an extra slash, ignore it - self.repo_slug = paths[1] + '/' + paths[2] + self.repo_slug = paths[1] + "/" + paths[2] self.pull_request_id = paths[4] end end end end