buildkite-builder 3.2.0 → 3.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d50f76bdcfda9a7561fce2b86dbae5b0ea99d37c0dbd6c763b35541a39e904c8
-  data.tar.gz: 2fbd91625db37b0bf88546931254b4f4024de25d3d20d09267318eafae2e3646
+  metadata.gz: 2946cdcde9aa6a6474bb3f9464d77b5ac9831fe1c5f3a27e289017e215f711a4
+  data.tar.gz: 6cbc69d2ea605f0427577daf28ed0243f56ccb3b12338662c08d377774ac5177
 SHA512:
-  metadata.gz: 5902b46da66c6ec7828d1bdf4f148f5445f2c7820e9f209e4a7ea22dae1ac8e62ea397660cca1f512737681c4c9397b97d5361f0fd64dba95066cb32751489ad
-  data.tar.gz: f67169e9bbdb7b84ccc1a6a8fec9593e878b70759e74f7236f595a3772649ce140d3ae99a329e090fb514c9bbbb450d70e58ea6362459404bced23f4a575cab1
+  metadata.gz: 04b63489255060c1237b8c25094b0aba3e99fe08079d23ab13173392e36908bf9db798224b70e5c28dfaccbb9323dfd2c11c8e4446bad0a9fd6fd4d74b44de8a
+  data.tar.gz: b1685985083035299d690371891f05d16c0dc932b159551f1b1ae23d645c4764d63456f44d29ce72e97251ad38ac6229622ba7795d173f8acc72a1ad09e8756e

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 3.3.0
+* Remove arguments in sub-pipeline's trigger step setup and use dsl delegation instead.
+
 ## 3.2.0
 * Remove `template` from sub-pipeline trigger step setup and use arguments instead.
 

data/VERSION

@@ -1 +1 @@
-3.2.0
+3.3.0

data/lib/buildkite/builder/extensions/sub_pipelines.rb

@@ -9,8 +9,15 @@ module Buildkite
 
           attr_reader :data, :name, :dsl
 
-          attribute :depends_on, append: true
+          # These attributes are for triggered step
+          attribute :label
           attribute :key
+          attribute :skip
+          attribute :if, as: :condition
+          attribute :depends_on, append: true
+          attribute :allow_dependency_failure
+          attribute :branches
+          attribute :async
 
           def self.to_sym
             name.split('::').last.downcase.to_sym
@@ -60,44 +67,35 @@ module Buildkite
         end
 
         dsl do
-          def pipeline(name, **options, &block)
+          def pipeline(name, &block)
             raise "Subpipeline must have a name" if name.empty?
             raise "Subpipeline does not allow nested in another Subpipeline" if context.is_a?(Buildkite::Builder::Extensions::SubPipelines::Pipeline)
 
             sub_pipeline = Buildkite::Builder::Extensions::SubPipelines::Pipeline.new(name, context, &block)
             context.data.pipelines.add(sub_pipeline)
 
-            options = options.slice(:key, :label, :async, :branches, :condition, :depends_on, :allow_dependency_failure, :skip, :emoji)
-            options[:key] ||= "subpipeline_#{name}_#{context.data.pipelines.count}"
-            options[:label] ||= name.capitalize
-
-            if options[:emoji]
-              emoji = Array(options.delete(:emoji)).map { |name| ":#{name}:" }.join
-              options[:label] = [emoji, options[:label]].compact.join(' ')
-            end
+            trigger_step = context.data.steps.add(Pipelines::Steps::Trigger)
+            trigger_step.trigger(name)
+            trigger_step.build(
+              message: '${BUILDKITE_MESSAGE}',
+              commit: '${BUILDKITE_COMMIT}',
+              branch: '${BUILDKITE_BRANCH}',
+              env: {
+                BUILDKITE_PULL_REQUEST: '${BUILDKITE_PULL_REQUEST}',
+                BUILDKITE_PULL_REQUEST_BASE_BRANCH: '${BUILDKITE_PULL_REQUEST_BASE_BRANCH}',
+                BUILDKITE_PULL_REQUEST_REPO: '${BUILDKITE_PULL_REQUEST_REPO}',
+                BKB_SUBPIPELINE_FILE: sub_pipeline.pipeline_yml
+              }
+            )
 
-            context.data.steps.add(Pipelines::Steps::Trigger, **options) do |context|
-              key context[:key]
-              label context[:label]
-              trigger name
-              build(
-                message: '${BUILDKITE_MESSAGE}',
-                commit: '${BUILDKITE_COMMIT}',
-                branch: '${BUILDKITE_BRANCH}',
-                env: {
-                  BUILDKITE_PULL_REQUEST: '${BUILDKITE_PULL_REQUEST}',
-                  BUILDKITE_PULL_REQUEST_BASE_BRANCH: '${BUILDKITE_PULL_REQUEST_BASE_BRANCH}',
-                  BUILDKITE_PULL_REQUEST_REPO: '${BUILDKITE_PULL_REQUEST_REPO}',
-                  BKB_SUBPIPELINE_FILE: sub_pipeline.pipeline_yml
-                }
-              )
-              async context[:async] || false
-              branches context[:branches] if context[:branches]
-              condition context[:condition] if context[:condition]
-              depends_on *context[:depends_on] if context[:depends_on]
-              allow_dependency_failure context[:allow_dependency_failure] || false
-              skip context[:skip] || false
-            end
+            trigger_step.key(sub_pipeline.key || "subpipeline_#{name}_#{context.data.pipelines.count}")
+            trigger_step.label(sub_pipeline.label || name.capitalize)
+            trigger_step.async(sub_pipeline.async || false)
+            trigger_step.branches(sub_pipeline.branches) if sub_pipeline.branches
+            trigger_step.condition(sub_pipeline.condition) if sub_pipeline.condition
+            trigger_step.depends_on(*sub_pipeline.get('depends_on')) if sub_pipeline.get('depends_on')
+            trigger_step.allow_dependency_failure(sub_pipeline.allow_dependency_failure || false)
+            trigger_step.skip(sub_pipeline.skip || false)
           end
         end
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-builder
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 3.3.0
 platform: ruby
 authors:
 - Ngan Pham
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-06-13 00:00:00.000000000 Z
+date: 2022-06-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rainbow