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

mountinfo: fix not showing package doc, typos #37

Merged
merged 2 commits into from
Sep 23, 2020

Conversation

kolyshkin
Copy link
Collaborator

No description provided.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Nothing from the doc.go was shown by godoc -- this is because the
comment about the package as a whole should precede the "package
$NAME", and this was broken here because of an empty line.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -173,7 +173,7 @@ func unescape(path string) (string, error) {
}

// The following code is UTF-8 transparent as it only looks for some
// specific characters (backslach and 0..7) with values < utf8.RuneSelf,
// specific characters (backslash and 0..7) with values < utf8.RuneSelf,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we enable misspell in golang-ci-lint? https://golangci-lint.run/usage/linters/

@thaJeztah thaJeztah merged commit 810cc47 into moby:master Sep 23, 2020
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.

None yet

2 participants