Skip to content

opts: FilterOpt: show empty string if no values are set - #6559

Merged
vvoland merged 1 commit into
docker:masterfrom
thaJeztah:fix_filteropts_default
Oct 13, 2025
Merged

opts: FilterOpt: show empty string if no values are set#6559
vvoland merged 1 commit into
docker:masterfrom
thaJeztah:fix_filteropts_default

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

follow-up to f81816e

We could decide to not initialize a default, and do this only when setting a value, but this may require more changes, so doing this as a starting-point.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

follow-up to f81816e
We could decide to not initialize a default, and do this only when
setting a value, but this may require more changes, so doing this
as a starting-point.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Files with missing linesPatch %Lines
opts/opts.go0.00%3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@vvoland
vvoland merged commit 68ec5bf into docker:masterOct 13, 2025
93 checks passed
@thaJeztah
thaJeztah deleted the fix_filteropts_default branch October 13, 2025 10:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@thaJeztah@codecov-commenter@vvoland@Benehiko