Skip to content

Conversation

@thaJeztah
Copy link
Member

It's not parsing anything, so we may as well inline it to be more clear what's done.

- 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)

@thaJeztah thaJeztah added this to the 29.1.3 milestone Dec 11, 2025
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Dec 11, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 11, 2025

Codecov Report

❌ Patch coverage is 72.22222% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/container/opts.go 0.00% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah force-pushed the inline_parseWindowsDevice branch from 85ae7d5 to 91d2271 Compare December 11, 2025 13:18
It's not parsing anything, so we may as well inline it to be more
clear what's done.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the inline_parseWindowsDevice branch from 91d2271 to 2e3425f Compare December 12, 2025 08:31
@thaJeztah thaJeztah changed the title cli/command/container: inline parseWindowsDevice cli/command/container: inline parseWindowsDevice, and minor cleanups Dec 12, 2025
@vvoland vvoland modified the milestones: 29.1.3, 29.2.0 Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants