Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Oct 15, 2020

No description provided.

@BYK BYK merged commit 76fbf02 into master Oct 15, 2020
@BYK BYK deleted the byk/ci/fix-gcb branch October 15, 2020 17:57
BYK added a commit that referenced this pull request Oct 15, 2020
This is a follow up to #21360 as it failed to address the underlying issue. GCB uses the `$xxx` syntax for variable expansion and it opreates in 'loose' mode when a trigger runs the build. This means it simply replaces unknown substitions with empty strings. This collides with how we try to use env variables which has the same syntax so the need to use `$$` syntax for GCB to not replace them with empty strings.
BYK added a commit that referenced this pull request Oct 15, 2020
This is a follow up to #21360 as it failed to address the underlying issue. GCB uses the `$xxx` syntax for variable expansion and it opreates in 'loose' mode when a trigger runs the build. This means it simply replaces unknown substitions with empty strings. This collides with how we try to use env variables which has the same syntax so the need to use `$$` syntax for GCB to not replace them with empty strings.
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2020
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.

3 participants