Skip to content
This repository has been archived by the owner on Oct 17, 2018. It is now read-only.

For standardization, use fmt.Fprintf(os.Stderr, ...) before logger available #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

For standardization, use fmt.Fprintf(os.Stderr, ...) before logger available #63

wants to merge 1 commit into from

Conversation

robskillington
Copy link
Contributor

No description provided.

xichen2020
xichen2020 approved these changes Oct 16, 2017
View reviewed changes
Copy link
Contributor

@xichen2020 xichen2020 left a comment

Choose a reason for hiding this comment

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

LGTM

@coveralls
Copy link

coveralls commented Oct 16, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 81.44% when pulling d86650f on r/use-golog-fatalf-in-place-of-printf-and-os-exit into 3541c54 on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@xichen2020 xichen2020 xichen2020 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@robskillington @coveralls @xichen2020