lib/danger/ci_source/buildkite.rb in danger-0.8.1 vs lib/danger/ci_source/buildkite.rb in danger-0.8.2

@@ -3,20 +3,26 @@
 
 module Danger
   module CISource
     class Buildkite < CI
       def self.validates?(env)
-        return !env["BUILDKITE"].nil?
+        return false unless env["BUILDKITE"]
+        return false unless env["BUILDKITE_REPO"]
+        return false unless env["BUILDKITE_PULL_REQUEST"]
+
+        return true
       end
 
       def initialize(env)
-        repo = env["BUILDKITE_REPO"]
-        unless repo.nil?
-          repo_matches = repo.match(%r{([\/:])([^\/]+\/[^\/.]+)(?:.git)?$})
-          self.repo_slug = repo_matches[2] unless repo_matches.nil?
-        end
-
+        self.repo_url = env["BUILDKITE_REPO"]
         self.pull_request_id = env["BUILDKITE_PULL_REQUEST"]
+
+        repo_matches = self.repo_url.match(%r{([\/:])([^\/]+\/[^\/.]+)(?:.git)?$})
+        self.repo_slug = repo_matches[2] unless repo_matches.nil?
+      end
+
+      def supported_request_sources
+        @supported_request_sources ||= [Danger::RequestSources::GitHub]
       end
     end
   end
 end