Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup clicktest output #1035

Merged
merged 3 commits into from
Feb 12, 2018

Conversation

ylecuyer
Copy link
Contributor

The clicktest output is too bloated. This is an attempt to clean it

@jung-kim
Copy link
Collaborator

jung-kim commented Feb 3, 2018

I think this is fine but I was hoping to use more proper login tools such as winston we are already using for this one.

@ylecuyer ylecuyer force-pushed the cleanup-clicktest-output branch 2 times, most recently from fd82410 to 44488fa Compare February 10, 2018 17:54
@ylecuyer
Copy link
Contributor Author

ylecuyer commented Feb 10, 2018

I updated the PR to use winston. if it's GTM for you let me know before so that I can squash the commits

@jung-kim jung-kim merged commit 30bdf2c into FredrikNoren:master Feb 12, 2018
@ylecuyer ylecuyer deleted the cleanup-clicktest-output branch March 22, 2018 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants