Skip to content

cli/command/plugin: fix linting issues, and assorted cleanups#6213

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:cleanup_plugins
Jul 28, 2025
Merged

cli/command/plugin: fix linting issues, and assorted cleanups#6213
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:cleanup_plugins

Conversation

@thaJeztah
Copy link
Member

  • fix various unhandled errors
  • remove some locally defined option-types in favor of option-types defined by the client / api
  • don't use unkeyed structs in tests, and add docs for some subtests
  • fix some values in tests that triggered "spellcheck" warnings
  • inline vars / functions that only had a single use.

- Human readable description for the release notes

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

@thaJeztah thaJeztah added this to the 29.0.0 milestone Jul 26, 2025
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Jul 26, 2025
@thaJeztah thaJeztah force-pushed the cleanup_plugins branch 3 times, most recently from 6ee5989 to 9509a09 Compare July 26, 2025 14:42
- fix various unhandled errors
- remove some locally defined option-types in favor of option-types
  defined by the client / api
- don't use unkeyed structs in tests, and add docs for some subtests
- fix some values in tests that triggered "spellcheck" warnings
- inline vars / functions that only had a single use.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah requested a review from Benehiko July 28, 2025 06:42
@thaJeztah thaJeztah merged commit a1035b0 into docker:master Jul 28, 2025
87 checks passed
@thaJeztah thaJeztah deleted the cleanup_plugins branch July 28, 2025 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants