Skip to content
This repository was archived by the owner on Apr 28, 2026. It is now read-only.

Only look for env vars that *start* with BUILDKITE#22

Open
glittershark wants to merge 1 commit into
Zegocover:masterfrom
glittershark:patch-1
Open

Only look for env vars that *start* with BUILDKITE#22
glittershark wants to merge 1 commit into
Zegocover:masterfrom
glittershark:patch-1

Conversation

@glittershark
Copy link
Copy Markdown

to:
cc: @zegocover/git-diff-conditional-buildkite-plugin-maintainers
related to:
resolves: #21

Rather than filtering env vars to pass to buildkite-pyyaml by those that
contain the string BUILDKITE, filter to variables that start with
BUILDKITE, using the -0 flag to env and the -z flag to grep to delimit
the variables by null characters rather than newlines so we don't also
grab lines inside multi-line env vars (such as the commit message)

Rather than filtering env vars to pass to buildkite-pyyaml by those that
*contain* the string BUILDKITE, filter to variables that *start* with
BUILDKITE, using the -0 flag to env and the -z flag to grep to delimit
the variables by null characters rather than newlines so we don't also
grab lines inside multi-line env vars (such as the commit message)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Commit messages containing the string "BUILDKITE" cause the command hook to fail

1 participant