Skip to content

assorted linting fixes and minor cleanups#259

Merged
AkihiroSuda merged 6 commits into
containerd:mainfrom
thaJeztah:fix_lint_step1
Oct 28, 2024
Merged

assorted linting fixes and minor cleanups#259
AkihiroSuda merged 6 commits into
containerd:mainfrom
thaJeztah:fix_lint_step1

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

    WARN [lintersdb] The name "vet" is deprecated. The linter has been renamed to: govet.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
    commands/mount_unsupported.go:26:31: var-declaration: should drop = nil from declaration of var MountCmd; it is the zero value (revive)
    var MountCmd *cobra.Command = nil
                                  ^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
ctx is commonly used for context.Context. Using it for any other purpose
should be avoided to prevent confusion.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@AkihiroSuda AkihiroSuda merged commit 0e83ada into containerd:main Oct 28, 2024
@thaJeztah thaJeztah deleted the fix_lint_step1 branch October 28, 2024 12:58
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