Skip to content

fix: Support []string fields in ReadFromEnvironment#3767

Merged
DrJosh9000 merged 1 commit intomainfrom
fix-readfromenvironment-does-not-support-slice
Mar 18, 2026
Merged

fix: Support []string fields in ReadFromEnvironment#3767
DrJosh9000 merged 1 commit intomainfrom
fix-readfromenvironment-does-not-support-slice

Conversation

@DrJosh9000
Copy link
Contributor

Description

Fix inability to update BUILDKITE_GIT_SUBMODULE_CLONE_CONFIG from a plugin or hook, and the accompanying warning log

Context

I noticed the following log:

2026/03/17 04:54:20 warning: job.ExecutorConfig.ReadFromEnvironment does not support slice for BUILDKITE_GIT_SUBMODULE_CLONE_CONFIG

This should fix it.

Changes

Check for slice kind, check the element type is string, split the value by comma, set the field value.

Testing

  • Tests have run locally (with go test ./...). Buildkite employees may check this if the pipeline has run automatically.
  • Code is formatted (with go tool gofumpt -extra -w .)

Disclosures / Credits

I did not use AI tools at all

@DrJosh9000 DrJosh9000 requested review from a team as code owners March 17, 2026 05:07
Copy link
Contributor

@zhming0 zhming0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@DrJosh9000 DrJosh9000 merged commit 878a037 into main Mar 18, 2026
2 checks passed
@DrJosh9000 DrJosh9000 deleted the fix-readfromenvironment-does-not-support-slice branch March 18, 2026 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants