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

fs/stat: add FreeBSD, and cleanup some nolint-comments #184

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

thaJeztah
Copy link
Member

relates to containerd/containerd#4579

this forward-ports the changes from containerd/containerd#4579 to this package, so that we can deprecate/remove the copy of these utilities from containerd (and recommend users to use the variant from this repository instead).

This forwards the build-tag from containerd/containerd:
containerd/containerd@4c47fe0

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Unlike comments for "humans", nolint comments should not have a leading
whitespace.

Also combined some comments, and renamed local variables so that we don't
have to suppress linting warnings for those.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit 10c7322 into containerd:main Jun 22, 2021
Copy link
Member

@samuelkarp samuelkarp left a comment

Choose a reason for hiding this comment

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

I'm late to seeing this, but LGTM.

@thaJeztah thaJeztah deleted the update_stat branch June 22, 2021 16:59
@thaJeztah
Copy link
Member Author

Thanks, Sam; thought it was "bsd" related, so possibly of interest to you 🤗

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.

4 participants