Skip to content

Add StrictPostBuildSubstitutions feature flag#1130

Merged
stefanprodan merged 5 commits intomainfrom
StrictPostBuildSubstitutions
Apr 9, 2024
Merged

Add StrictPostBuildSubstitutions feature flag#1130
stefanprodan merged 5 commits intomainfrom
StrictPostBuildSubstitutions

Conversation

@stefanprodan
Copy link
Copy Markdown
Member

@stefanprodan stefanprodan commented Apr 8, 2024

Changes:

Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan added enhancement New feature or request area/varsub Post-build variable substitution related issues and pull requests labels Apr 8, 2024
@stefanprodan stefanprodan requested a review from souleb April 8, 2024 16:11
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan force-pushed the StrictPostBuildSubstitutions branch from 2a9eceb to 76cc820 Compare April 9, 2024 07:02
Ref: #1082

Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
@stefanprodan stefanprodan force-pushed the StrictPostBuildSubstitutions branch from a7e2111 to b810013 Compare April 9, 2024 07:11
Copy link
Copy Markdown
Member

@souleb souleb left a comment

Choose a reason for hiding this comment

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

LGTM!

@stefanprodan stefanprodan merged commit fa5cebb into main Apr 9, 2024
@stefanprodan stefanprodan deleted the StrictPostBuildSubstitutions branch April 9, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/varsub Post-build variable substitution related issues and pull requests enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to error on missing postBuild substitution variable var substitution failed, found unknown escape character

2 participants