Skip to content

Conversation

brendanashworth
Copy link
Contributor

Only objects and symbols use util.inspect, others are simply
concatenated.

Fixes: #935
R=@cjihrig?

@mscdex
Copy link
Contributor

mscdex commented Mar 25, 2015

Shouldn't this target the doc subsystem?

@mscdex mscdex added the doc Issues and PRs related to the documentations. label Mar 25, 2015
@cjihrig
Copy link
Contributor

cjihrig commented Mar 25, 2015

LGTM, but as pointed out, please change the commit subsystem to doc.

Only objects and symbols use `util.inspect`, others are simply
concatenated.

Fixes: nodejs#935
@brendanashworth brendanashworth changed the title util: fix format docs discrepancy doc: fix format docs discrepancy Mar 25, 2015
@brendanashworth
Copy link
Contributor Author

Ah, yes, thank you, I missed that. It's now been fixed, merging in a short while.

brendanashworth added a commit that referenced this pull request Mar 25, 2015
Only objects and symbols use `util.inspect`, others are simply
concatenated.

Fixes: #935
PR-URL: #1255
Reviewed-By: Colin Ihrig <[email protected]>
@brendanashworth
Copy link
Contributor Author

Thanks, this has been merged in 4581421.

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

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants