Skip to content

daemon/graphdriver/devmapper/ fix lint errors/warnings#14930

Merged
tiborvass merged 1 commit intomoby:masterfrom
brahmaroutu:lint_daemon_graphdriver_devmapper
Jul 29, 2015
Merged

daemon/graphdriver/devmapper/ fix lint errors/warnings#14930
tiborvass merged 1 commit intomoby:masterfrom
brahmaroutu:lint_daemon_graphdriver_devmapper

Conversation

@brahmaroutu
Copy link
Contributor

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

Apart from making golint run, I have changed many variables and methods to local. Hope this devmapper pakage(not under pkg) is not used outside of docker code base. Please review.

Copy link
Member

Choose a reason for hiding this comment

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

s/usagei/usage/
s/used/and is used/

Copy link
Contributor

Choose a reason for hiding this comment

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

this description seems unfinished.

@brahmaroutu brahmaroutu force-pushed the lint_daemon_graphdriver_devmapper branch from bf9dece to 3787f5a Compare July 28, 2015 19:12
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this capitalized?

@LK4D4
Copy link
Contributor

LK4D4 commented Jul 28, 2015

@brahmaroutu Also need rebase :/

@brahmaroutu brahmaroutu force-pushed the lint_daemon_graphdriver_devmapper branch from 3787f5a to 1fb052f Compare July 28, 2015 22:42
@RichardScothern
Copy link
Contributor

LGTM. Thanks @brahmaroutu !

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.

9 participants