buildkite-builder 4.5.0 → 4.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c47c9c53151cf213174d4be297b48cc832a92d112c0227eea50ee04f7b8a2951
-  data.tar.gz: 1d2db37e56f3e5a5dbe13b2cb5b6355fe58a698ef60197ad4bd7d16d07d347cf
+  metadata.gz: 483d18ac586fc6073d55347f1385ed50741046001c677201f4a6967b1ecf2357
+  data.tar.gz: 5ffa487ec9034858aed6f1e23df31caa291ddfd49c8410bf42e2cde61568d3f9
 SHA512:
-  metadata.gz: 256e662ade3a59a9ed2660b8e954781c3cbe995476187e8c186eadbe9e35a0e9c41ac750e5df8afc932c2643df59b99c51bb35b6dc878cddd7f17580e41b17ba
-  data.tar.gz: 40d109033e53bfc4eb80a4bfb9721cd5569d7fe9c608962da1e9003a0d202c1ff3d3edd7b89f1b6239ba3c7a3108513733abd898f1983491c12dc2fdb8c4d528
+  metadata.gz: ffbe3b03a00d903939eba8a21d051c56ceccc910cf0477ada10f14fdddd5ebbbfacf74157aefcf4fdbf4487de46cb4619ddeccc215760dcde665684d33c79d47
+  data.tar.gz: 610ae8d5bffc2c413a65ae8083694f619e4ee312914b54de53d486400fb25416ddbdda39334b42fd57d6b8208396570923ae91b9b67935a762c0719ebf9a4e58

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+### 4.6.0
+* Remove `capture` concept on `Buildkite::Pipelines::Command#run` and replaced with `Buildkite::Pipelines::Command::Result` object to represent `Open3.capture3` result.
+
 ### 4.5.0
 * Do not upload `pipeline.yml` when steps is empty.
 

data/VERSION

@@ -1 +1 @@
-4.5.0
+4.7.0

data/lib/buildkite/builder/commands/run.rb

@@ -17,7 +17,7 @@ module Buildkite
           # variables to be set. It also uploads the pipeline to Buildkite.
           log.info "+++ 🧰 #{'Buildkite Builder'.color(:springgreen)} v#{Buildkite::Builder.version} ─ #{relative_pipeline_path.to_s.yellow}"
 
-          if Buildkite::Pipelines::Command.meta_data(:exists, Builder.meta_data.fetch(:job))
+          if Buildkite::Pipelines::Command.meta_data(:exists, Builder.meta_data.fetch(:job)).success?
             log.info "Pipeline already uploaded in #{Buildkite.env.step_id} step".color(:dimgray)
           else
             Pipeline.new(pipeline_path, logger: log).upload

data/lib/buildkite/builder/extension.rb

@@ -4,8 +4,6 @@ module Buildkite
   module Builder
     class Extension
       class << self
-        attr_reader :dsl
-
         def dsl(&block)
           @dsl = Module.new(&block) if block_given?
           @dsl

data/lib/buildkite/builder/pipeline.rb

@@ -54,8 +54,11 @@ module Buildkite
             file.write(contents)
 
             logger.info "+++ :pipeline: Uploading pipeline"
-            unless Buildkite::Pipelines::Command.pipeline(:upload, file.path)
-              logger.info "Pipeline upload failed, saving as artifact…"
+
+            result = Buildkite::Pipelines::Command.pipeline(:upload, file.path)
+
+            unless result.success?
+              logger.info "Pipeline upload failed with #{result.stderr}, saving as artifact…"
               Buildkite::Pipelines::Command.artifact!(:upload, file.path)
               abort
             end

data/lib/buildkite/pipelines/command.rb

@@ -7,6 +7,23 @@ module Buildkite
     class Command
       class CommandFailedError < StandardError; end
 
+      class Result
+        attr_reader :stdout, :stderr
+        def initialize(stdout, stderr, status)
+          @stdout = stdout.strip
+          @stderr = stderr.strip
+          @status = status
+        end
+
+        def success?
+          @status.success?
+        end
+
+        def output
+          @output ||= "#{stdout}\n#{stderr}".strip
+        end
+      end
+
       BIN_PATH = 'buildkite-agent'
       COMMANDS = %w(
         pipeline
@@ -21,12 +38,12 @@ module Buildkite
         end
 
         def artifact(subcommand, *args, exception: false)
-          capture = case subcommand.to_s
-          when 'shasum', 'search' then true
-          else false
-          end
+          result = new(:artifact, subcommand, *args).run(exception: exception)
 
-          new(:artifact, subcommand, *args).run(capture: capture, exception: exception)
+          case subcommand.to_s
+          when 'shasum', 'search' then result.output
+          else result
+          end
         end
 
         def annotate(body, *args, exception: false)
@@ -34,12 +51,12 @@ module Buildkite
         end
 
         def meta_data(subcommand, *args, exception: false)
-          capture = case subcommand.to_s
-          when 'get', 'keys' then true
-          else false
-          end
+          result = new(:'meta-data', subcommand, *args).run(exception: exception)
 
-          new(:'meta-data', subcommand, *args).run(capture: capture, exception: exception)
+          case subcommand.to_s
+          when 'get', 'keys' then result.output
+          else result
+          end
         end
       end
 
@@ -58,16 +75,14 @@ module Buildkite
         @args = transform_args(args)
       end
 
-      def run(capture: false, exception: false)
+      def run(exception: false)
         stdout, stderr, status = Open3.capture3(*to_a)
-        if capture
-          stdout
-        elsif status.success?
-          true
-        elsif exception
-          raise CommandFailedError, "#{stdout}\n#{stderr}"
+        result = Result.new(stdout, stderr, status)
+
+        if !result.success? && exception
+          raise CommandFailedError, "#{result.output}"
         else
-          false
+          result
         end
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-builder
 version: !ruby/object:Gem::Version
-  version: 4.5.0
+  version: 4.7.0
 platform: ruby
 authors:
 - Ngan Pham
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-10-31 00:00:00.000000000 Z
+date: 2024-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rainbow
@@ -174,7 +174,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.13
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: A gem for programmatically creating Buildkite pipelines.