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.

Sign up for GitHub

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

Jump to bottom

%o is not documented in Readme.md #302

Closed
rafis opened this issue Aug 28, 2016 · 2 comments
Closed

%o is not documented in Readme.md #302

rafis opened this issue Aug 28, 2016 · 2 comments
Labels
has-pr This issue has an accompanying pull request

Comments

@rafis
Copy link

rafis commented Aug 28, 2016 •
edited

'%o' is not documented. Or even better would be to not to pass arguments to util.inspect directly, but pass them through own formatter depending on type of argument, if it is an Object or Array then format using formatter.o:

debug('myObj=', myObj); // doesn't work because '%o' is not documented
debug('myObj=%o', myObj); // works because '%o' is now documented and people knows about it
@thebigredgeek
Copy link
Contributor

@rafis would you like to create a PR for this?

@thebigredgeek thebigredgeek mentioned this issue Nov 14, 2016
Merged
@thebigredgeek
Copy link
Contributor

Docs should be added here

@thebigredgeek thebigredgeek added the has-pr This issue has an accompanying pull request label Nov 14, 2016
TooTallNate added a commit that referenced this issue Dec 12, 2016
@TooTallNate
Readme: add Formatters section
253f9dd 
Fixes #302.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
has-pr This issue has an accompanying pull request
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thebigredgeek @rafis