-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Rename inspect
to audit
#1505
Labels
help wanted
Issues we wouldn't mind assistance with.
Comments
benlesh
added
help wanted
Issues we wouldn't mind assistance with.
priority: high
labels
Mar 21, 2016
Closed via #1512. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Relates to discussed issue here #1387. Basically,
inspect
is a name-collision in Node because it's an API that should have been aSymbol
, but couldn't have been because it predates Symbols. /shrugAnyhow, since the
inspect
operator is brand new with this major version, we can rename it. I thinkaudit
is a fine name, props to @kennethlawrence for the idea.Labeling priority
high
because we don't want any breaking changes to existing APIs after production release.The text was updated successfully, but these errors were encountered: