Merge pull request #147 from skipkayhil/hm-orlspzouxytyluwl

Automatically ci-skip if all changed files are md

Author: rafaelfranca

bin/fetch-pr

@@ -17,7 +17,14 @@ def github_repo
   ENV.fetch("BUILDKITE_REPO")[%r{github\.com[/:](.+?)(?:\.git)?\z}, 1]
 end
 
-uri = URI("https://api.github.com/repos/#{github_repo}/pulls/#{pull_request_number}")
+case ARGV
+in ["files"]
+  $stderr.write "Fetching pull-request files: "
+  uri = URI("https://api.github.com/repos/#{github_repo}/pulls/#{pull_request_number}/files")
+else
+  $stderr.write "Fetching pull-request metadata: "
+  uri = URI("https://api.github.com/repos/#{github_repo}/pulls/#{pull_request_number}")
+end
 
 req = Net::HTTP::Get.new(uri)
 req["Authorization"] = "token #{github_token}"
@@ -26,4 +33,6 @@ res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: true) do |http|
   http.request(req)
 end
 
+$stderr.write "#{res.code}.\n"
+
 puts res.body

lib/buildkite/config/build_context.rb

@@ -23,11 +23,22 @@ def nightly?
       ENV.has_key?("RAILS_CI_NIGHTLY")
     end
 
+    def full?
+      # [ci full], [full ci], [ci-full], or [full-ci]
+      [ENV["BUILDKITE_MESSAGE"], FetchPr.title].grep(/(ci full|full ci|ci-full|full-ci)/i).any?
+    end
+
     def skip?
       # [ci skip], [skip ci], [ci-skip], or [skip-ci]
       [ENV["BUILDKITE_MESSAGE"], FetchPr.title].grep(/(ci skip|skip ci|ci-skip|skip-ci)/i).any?
     end
 
+    def only_markdown?
+      filenames = FetchPr.filenames
+
+      !filenames.empty? && filenames.all? { |name| name.end_with? ".md" }
+    end
+
     def rails_root
       Pathname.pwd
     end

lib/buildkite/config/fetch_pr.rb

@@ -4,11 +4,21 @@
 
 module Buildkite::Config
   module FetchPr
-    def self.title
-      pr = JSON.parse(File.read(".buildkite/tmp/.pr-meta.json"))
-      pr["title"]
-    rescue
-      ""
+    class << self
+      def title
+        pr = JSON.parse(File.read(".buildkite/tmp/.pr-meta.json"))
+        pr["title"]
+      rescue
+        ""
+      end
+
+      def filenames
+        JSON
+          .load_file(".buildkite/tmp/.pr-files.json")
+          .map { |f| f["filename"] }
+      rescue
+        []
+      end
     end
   end
 end

pipelines/rails-ci/initial.yml

@@ -27,7 +27,6 @@ steps:
 
       rm -rf .buildkite/.git
 
-      echo "Fetching pull-request metadata:"
       (docker run --rm \
         -v "$$PWD":/app:ro -w /app \
         -e GITHUB_PUBLIC_REPO_TOKEN \
@@ -36,6 +35,14 @@ steps:
         ruby:latest \
         .buildkite/bin/fetch-pr > .buildkite/tmp/.pr-meta.json) || true
 
+      (docker run --rm \
+        -v "$$PWD":/app:ro -w /app \
+        -e GITHUB_PUBLIC_REPO_TOKEN \
+        -e BUILDKITE_REPO \
+        -e BUILDKITE_PULL_REQUEST \
+        ruby:latest \
+        .buildkite/bin/fetch-pr files > .buildkite/tmp/.pr-files.json) || true
+
       echo "Generating pipeline:"
       sh -c "$$PIPELINE_COMMAND"
 

pipelines/rails-ci/pipeline.rb

@@ -42,7 +42,7 @@
     end
   end
 
-  if build_context.skip?
+  if !build_context.full? && (build_context.skip? || build_context.only_markdown?)
     command do
       label ":bk-status-passed: Build skipped"
       skip true