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

Improved logging #4

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

Conversation

thelinuxlich
Copy link

  • options.logLevel can be a string, where its value must be 'debug', 'verbose', 'info', 'warn', or 'error'
  • options.log can be an object with {debug, verbose, info, warn, error} methods
  • When options.log is a function, its second argument will be the log level of the given message
  • When options.log is undefined, the console methods are used by default
  • When options.log is falsy (or options.silent is truthy), log messages are silenced
  • Some log messages have been tweaked, and log levels attached
  • The pool master now emits an 'error' event with the associated Error object

- `options.logLevel` can be a string, where its value must be 'debug', 'verbose', 'info', 'warn', or 'error'
- `options.log` can be an object with {debug, verbose, info, warn, error} methods
- When `options.log` is a function, its second argument will be the log level of the given message
- When `options.log` is undefined, the `console` methods are used by default
- When `options.log` is falsy (or `options.silent` is truthy), log messages are silenced
- Some log messages have been tweaked, and log levels attached
- The pool master now emits an 'error' event with the associated `Error` object
@thelinuxlich
Copy link
Author

@aleclarson are you willing to rebase your work on our fork?

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