Skip to content

fix(run-integration-test): Correctly parse config options using yq#22

Merged
Techassi merged 4 commits into
mainfrom
fix/run-integration-test-yq-parsing
Dec 6, 2024
Merged

fix(run-integration-test): Correctly parse config options using yq#22
Techassi merged 4 commits into
mainfrom
fix/run-integration-test-yq-parsing

Conversation

@Techassi
Copy link
Copy Markdown
Member

@Techassi Techassi commented Dec 6, 2024

Part of stackabletech/issues#659

This now correctly passes options from the config file while also falling back to default values if not set.

@Techassi Techassi self-assigned this Dec 6, 2024
@Techassi Techassi marked this pull request as ready for review December 6, 2024 12:13
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@Techassi Techassi merged commit a6da884 into main Dec 6, 2024
@Techassi Techassi deleted the fix/run-integration-test-yq-parsing branch December 6, 2024 12:21
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