Skip to content

Various small fix-ups#202

Merged
dmcgowan merged 4 commits into
containerd:mainfrom
thaJeztah:various_fixups
Apr 5, 2022
Merged

Various small fix-ups#202
dmcgowan merged 4 commits into
containerd:mainfrom
thaJeztah:various_fixups

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

fs: fix minor linting and gofmt issue

sysx: remove unused sysx/generate.sh script

This script was added in 0817db4, but the code it generated was upstreamed to golang.org/x/sys, and replaced in 6268e28, so should no longer be needed.

golangci-lint: replace "golint" with "revive"

The linter 'golint' is deprecated (since v1.41.0) due to: The repository of the linter has been archived by the owner. Replaced by revive.

README: update badges and links

The Build Status badge was still linked to travis, which we no longer use. While at it, also updated the links to the documentation, and updated a link to the runtime-spec.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This script was added in 0817db4,
but the code it generated was upstreamed to golang.org/x/sys, and
replaced in 6268e28, so should no
longer be needed.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
The linter 'golint' is deprecated (since v1.41.0) due to: The repository
of the linter has been archived by the owner.  Replaced by revive.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
The Build Status badge was still linked to travis, which we no longer
use. While at it, also updated the links to the documentation, and
updated a link to the runtime-spec.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah changed the title Various fixups Various small fix-ups Apr 4, 2022
@thaJeztah thaJeztah requested a review from estesp April 4, 2022 16:46
@dmcgowan dmcgowan merged commit 164806d into containerd:main Apr 5, 2022
@thaJeztah thaJeztah deleted the various_fixups branch April 5, 2022 17:29
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.

3 participants