Skip to content

Add docstring to reexec.Command#14762

Merged
duglin merged 1 commit intomoby:masterfrom
LK4D4:lint_reexec
Jul 21, 2015
Merged

Add docstring to reexec.Command#14762
duglin merged 1 commit intomoby:masterfrom
LK4D4:lint_reexec

Conversation

@LK4D4
Copy link
Contributor

@LK4D4 LK4D4 commented Jul 20, 2015

No description provided.

@estesp
Copy link
Contributor

estesp commented Jul 20, 2015

LGTM

Copy link
Member

Choose a reason for hiding this comment

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

Also it setting could be removed here for windows right?

@duglin
Copy link
Contributor

duglin commented Jul 20, 2015

Aside from small typo found by @runcom, LGTM

Signed-off-by: Alexander Morozov <lk4d4@docker.com>
@LK4D4
Copy link
Contributor Author

LK4D4 commented Jul 21, 2015

@runcom @duglin fixed

@runcom
Copy link
Member

runcom commented Jul 21, 2015

LGTM waiting janky

duglin added a commit that referenced this pull request Jul 21, 2015
Add docstring to reexec.Command
@duglin duglin merged commit 4d0990a into moby:master Jul 21, 2015
@LK4D4 LK4D4 deleted the lint_reexec branch July 21, 2015 02:38
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