Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

fix(test): fix config tests #409

Merged
merged 3 commits into from
Sep 26, 2023
Merged

fix(test): fix config tests #409

merged 3 commits into from
Sep 26, 2023

Conversation

RogerLamTd
Copy link
Contributor

config tests would not register values after any bool flag was set with "-" + flags.xyz.Name, "true". Switch to "--" syntax and add necessary checks in app.Action() so they're actually checked.

@davidtaikocha davidtaikocha enabled auto-merge (squash) September 26, 2023 05:13
@davidtaikocha davidtaikocha merged commit 4f0a602 into main Sep 26, 2023
4 checks passed
@davidtaikocha davidtaikocha deleted the fix-config-tests branch September 26, 2023 05:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants