Skip to content

daemon/graphdriver/zfs fix lint errrors/warnings#15269

Merged
icecrime merged 1 commit intomoby:masterfrom
brahmaroutu:lint_daemon_graphdriver_zfs
Aug 5, 2015
Merged

daemon/graphdriver/zfs fix lint errrors/warnings#15269
icecrime merged 1 commit intomoby:masterfrom
brahmaroutu:lint_daemon_graphdriver_zfs

Conversation

@brahmaroutu
Copy link
Contributor

Addresses #14756
Signed-off-by: Srini Brahmaroutu srbrahma@us.ibm.com

Not sure if this package is already listed or not.

@calavera
Copy link
Contributor

calavera commented Aug 3, 2015

I think it needs to be added to the list in hack/make/validate-lint.

Copy link
Member

Choose a reason for hiding this comment

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

s/wraps/wrapps/

Addresses moby#14756
Signed-off-by: Srini Brahmaroutu <srbrahma@us.ibm.com>
@brahmaroutu brahmaroutu force-pushed the lint_daemon_graphdriver_zfs branch from 06f95fb to e27c904 Compare August 4, 2015 18:31
@brahmaroutu
Copy link
Contributor Author

@vdemeester Update as per your comments.

@calavera
Copy link
Contributor

calavera commented Aug 4, 2015

LGTM

1 similar comment
@icecrime
Copy link
Contributor

icecrime commented Aug 5, 2015

LGTM

icecrime pushed a commit that referenced this pull request Aug 5, 2015
daemon/graphdriver/zfs fix lint errrors/warnings
@icecrime icecrime merged commit 2c3cd94 into moby:master Aug 5, 2015
@brahmaroutu brahmaroutu deleted the lint_daemon_graphdriver_zfs branch August 12, 2015 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants