Log targets : A minimal remediation for Issue #4 #5

Merged
merged 2 commits into from Aug 4, 2013

Projects

None yet

3 participants

@lorefnon
lorefnon commented Aug 1, 2013

No description provided.

@ericclemmons ericclemmons merged commit c3e36be into ericclemmons:master Aug 4, 2013

1 check passed

Details default The Travis CI build passed
@rearadmiral

Could you add a flag for this logging?

Maybe only log if verbose: true or offer a quiet: true override.

It's a bit noisy and I usually don't want to see it.

Happy to create a PR if you accept the change but don't have time to implement.

@ericclemmons
Owner

@rearadmiral I agree, this is verbose. Can you submit a PR for verbose: true (verbosity should be opted into, IMO)? I'm short on time nowadays :(

Thanks!

@rearadmiral

Sounds good. Split into #51

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