Skip to content

The logName is confused here, it's a localName actually.#18661

Merged
vdemeester merged 1 commit intomoby:masterfrom
zhanghuanzhong:docker-fix-a-confused-variable
Dec 15, 2015
Merged

The logName is confused here, it's a localName actually.#18661
vdemeester merged 1 commit intomoby:masterfrom
zhanghuanzhong:docker-fix-a-confused-variable

Conversation

@zhanghuanzhong
Copy link
Contributor

As the title said.
Signed-off-by: huanzhong zhang [email protected]

@thaJeztah
Copy link
Member

I suspect this was named logName, because it's the name used in the log-message. Guess this works as well, so

LGTM

@tiborvass
Copy link
Contributor

LGTM

@vdemeester
Copy link
Member

LGTM 🐻
Windows build are failing for nothing 😅 merging 😉

vdemeester added a commit that referenced this pull request Dec 15, 2015
…ariable

The logName is confused here, it's a localName actually.
@vdemeester vdemeester merged commit 7fe5631 into moby:master Dec 15, 2015
@zhanghuanzhong
Copy link
Contributor Author

Thanks for your review :)

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