Merge branch 'trunk' into update/buildkite-test-collector

Author: spencertransier

CHANGELOG.md

@@ -24,6 +24,7 @@ _None_
 - Update `octokit` to `5.6.1` This is a major version bump from version `4.18`, but is not a breaking change for the Release Toolkit because it doesn't change any public APIs for clients. [#464]
 - Update `danger` to `9.3.0`. This is an internal-only change and is not a breaking change for clients. [#464]
 - Replace `rspec-buildkite-analytics` with `buildkite-test_collector` (Buildkite renamed the gem) and update it to `2.2.0`. This is another internal-only change and is not a breaking change for clients. [#465]
+- Adds `ignore_pipeline_branch_filters=true` parameter to the API call triggering a Buildkite build [#468]
 
 ## 7.0.0
 

lib/fastlane/plugin/wpmreleasetoolkit/actions/common/buildkite_trigger_build_action.rb

@@ -13,7 +13,11 @@ def self.run(params)
           branch: params[:branch],
           commit: params[:commit],
           env: params[:environment].merge(pipeline_name),
-          message: params[:message]
+          message: params[:message],
+          # Buildkite will not trigger a build if the GitHub activity for that branch is turned off
+          # We want API triggers to work regardless of the GitHub activity settings, so this option is necessary
+          # https://forum.buildkite.community/t/request-build-error-branches-have-been-disabled-for-this-pipeline/1463/2
+          ignore_pipeline_branch_filters: true
         }.compact # remove entries with `nil` values from the Hash, if any
 
         client = Buildkit.new(token: params[:buildkite_token])