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

Unify error logging #82

Merged
merged 2 commits into from
Jul 11, 2016
Merged

Unify error logging #82

merged 2 commits into from
Jul 11, 2016

Conversation

dwbutler
Copy link
Owner

Internal refactoring to make all error logging go through an error logger. The default error logger logs to STDERR, but a custom error logger can be provided by passing a new error_logger option or by changing LogStashLogger.configuration.default_error_logger.

Internal refactoring to make all error logging go through an error logger. The default error logger logs to STDERR, but a custom error logger can be provided by passing a new `error_logger` option or by changing `LogStashLogger.configuration.default_error_logger`.
@codecov-io
Copy link

codecov-io commented Jul 10, 2016

Current coverage is 89.91%

Merging #82 into master will decrease coverage by 0.23%

@@             master        #82   diff @@
==========================================
  Files            55         55          
  Lines          1229       1240    +11   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1108       1115     +7   
- Misses          121        125     +4   
  Partials          0          0          

Powered by Codecov. Last updated by fb86f42...93f7a7c

@dwbutler dwbutler merged commit 9ad139a into master Jul 11, 2016
@dwbutler dwbutler deleted the feature/error-logger branch July 11, 2016 00:00
@dwbutler
Copy link
Owner Author

Released in 0.16.0.

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.

2 participants