-
Notifications
You must be signed in to change notification settings - Fork 81
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
Logger function in options #452
Logger function in options #452
Conversation
The config object passed as a param to the function `setup.compressed` was incomplete. I added a noop as the info logging function to make it green.
My bad: I didn't write documentation for this. Should I do that? |
yes please! documentation would be great |
you'll want to pull down the changes I just made to this... I cleaned up some conflicts in the code. |
Code looks great, just waiting on some documentation to get this merged. Thanks so much! @dottorblaster |
@designfrontier just pushed my README updates in fc2fd33, hope this is ok 👍 |
Strong Work! This looks fantastic. |
Thanks! |
As per #444, I'm implementing a pluggable logger with defaults going to trigger simple console invokes.
I updated tests too, hope this helps!