Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various small fix-ups #202

Merged
merged 4 commits into from
Apr 5, 2022
Merged

Various small fix-ups #202

merged 4 commits into from
Apr 5, 2022

Conversation

thaJeztah
Copy link
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