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

doc: fix typo #8107

Closed
wants to merge 1 commit into from
Closed

doc: fix typo #8107

wants to merge 1 commit into from

Conversation

bengourley
Copy link
Contributor

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

None, just a tiny doc change

Description of change

Fix a typo in the querystring docs: obj.hashOwnProperty()obj.hasOwnProperty().

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. querystring Issues and PRs related to the built-in querystring module. labels Aug 15, 2016
@targos
Copy link
Member

targos commented Aug 15, 2016

LGTM

@princejwesley
Copy link
Contributor

princejwesley commented Aug 15, 2016

LGTM.
Commit message suggestion: doc: fix "hashOwnProperty" typo in querystring (First line should not exceed 50 columns)

@mscdex
Copy link
Contributor

mscdex commented Aug 15, 2016

LGTM

querystring subsystem docs referred to `obj.hashOwnProperty()`
which is non-existent. Corrected to `obj.hasOwnProperty()`.
@bengourley
Copy link
Contributor Author

@princejwesley I've amended the commit message per your recommendation.

@JungMinu
Copy link
Member

LGTM

@Trott
Copy link
Member

Trott commented Aug 15, 2016

I think this definitely qualifies for the "trivial change" exception to the "wait 48 hours" rule so I'm going to ahead and land it right now...

Trott pushed a commit to Trott/io.js that referenced this pull request Aug 15, 2016
querystring subsystem docs referred to `obj.hashOwnProperty()`
which is non-existent. Corrected to `obj.hasOwnProperty()`.

PR-URL: nodejs#8107
Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
@Trott
Copy link
Member

Trott commented Aug 15, 2016

Landed in 24e4488. Thanks for the fix!

@Trott Trott closed this Aug 15, 2016
evanlucas pushed a commit that referenced this pull request Aug 20, 2016
querystring subsystem docs referred to `obj.hashOwnProperty()`
which is non-existent. Corrected to `obj.hasOwnProperty()`.

PR-URL: #8107
Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
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. querystring Issues and PRs related to the built-in querystring module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants