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: add data types to properties #9399

Closed
cjihrig opened this issue Nov 1, 2016 · 9 comments
Closed

doc: add data types to properties #9399

cjihrig opened this issue Nov 1, 2016 · 9 comments
Labels
doc Issues and PRs related to the documentations.

Comments

@cjihrig
Copy link
Contributor

cjihrig commented Nov 1, 2016

Some of the docs specify the type for properties, while others do not. A good first contribution would be to update the properties that are missing this information. As an example, look at https://nodejs.org/api/process.html#process_process_connected vs. https://nodejs.org/api/child_process.html#child_process_child_connected. These two things represent the same concept, but the one in the child_process docs have the data type.

@cjihrig cjihrig added good first issue Issues that are suitable for first-time contributors. docs-requested labels Nov 1, 2016
@JungMinu JungMinu self-assigned this Nov 2, 2016
@tanujasawant
Copy link
Contributor

@JungMinu Are you still planning on working on this issue? I am an Outreachy applicant and a newbie to Open Source. I have been visiting this issue with utmost excitement to see its outcome. Please comment if you're no longer interested.
Thanks!

@Trott
Copy link
Member

Trott commented Nov 3, 2016

I would request that people who already have contributions in the project please refrain from reserving new good first contribution issues. It makes the label considerably less useful for people looking for their actual first contribution.

(It's a bit different for a good first contribution issue that has been open for a while. If a good first contribution issue has been open for a few weeks, it's probably appropriate to remove the good first contribution label and submit a PR. But that's not the case here.)

@tanujasawant
Copy link
Contributor

@Trott Forgive me, I was unaware of the fact that people might still be looking for first contributions. I'll look for other issues. Thanks!

@JungMinu JungMinu removed their assignment Nov 4, 2016
@JungMinu
Copy link
Member

JungMinu commented Nov 4, 2016

@Trott Sorry, it was a mistake. I've just removed self-assignment

@imatveev
Copy link
Contributor

imatveev commented Nov 7, 2016

Added most of values in pull-request

@imatveev
Copy link
Contributor

imatveev commented Nov 9, 2016

Yet another pull-request

@imatveev
Copy link
Contributor

Yet another pull-request

@sam-github sam-github added doc Issues and PRs related to the documentations. and removed doc Issues and PRs related to the documentations. docs-requested labels Dec 1, 2016
@targos
Copy link
Member

targos commented Jan 8, 2017

Are there still missing types?

ameliavoncat added a commit to ameliavoncat/node that referenced this issue Feb 25, 2017
ameliavoncat added a commit to ameliavoncat/node that referenced this issue Feb 25, 2017
ameliavoncat added a commit to ameliavoncat/node that referenced this issue Mar 11, 2017
italoacasas pushed a commit to italoacasas/node that referenced this issue Mar 13, 2017
Ref: nodejs#9399
PR-URL: nodejs#11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this issue Mar 14, 2017
Ref: nodejs#9399
PR-URL: nodejs#11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
ameliavoncat added a commit to ameliavoncat/node that referenced this issue Mar 14, 2017
jasnell pushed a commit that referenced this issue Mar 15, 2017
Refs: #9399
PR-URL: #11802
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Brian White <[email protected]>
jasnell pushed a commit that referenced this issue Mar 15, 2017
Refs: #9399
PR-URL: #11799
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
jasnell pushed a commit that referenced this issue Mar 15, 2017
Refs: #9399
PR-URL: #11764
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this issue Mar 20, 2017
Refs: nodejs#9399
PR-URL: nodejs#11802
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Brian White <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this issue Mar 20, 2017
Refs: nodejs#9399
PR-URL: nodejs#11764
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
Ref: nodejs#9399
PR-URL: nodejs#11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
Ref: nodejs#9399
PR-URL: nodejs#11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
Refs: nodejs#9399
PR-URL: nodejs#11802
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Brian White <[email protected]>
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
Refs: nodejs#9399
PR-URL: nodejs#11799
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
Refs: nodejs#9399
PR-URL: nodejs#11764
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
gibfahn pushed a commit that referenced this issue Jun 17, 2017
Ref: #9399
PR-URL: #11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
gibfahn pushed a commit that referenced this issue Jun 20, 2017
Ref: #9399
PR-URL: #11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
MylesBorins pushed a commit that referenced this issue Jul 11, 2017
Ref: #9399
PR-URL: #11681
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
@Trott
Copy link
Member

Trott commented Jul 15, 2017

This issue has been inactive for sufficiently long that it seems like perhaps it should be closed. Feel free to re-open (or leave a comment requesting that it be re-opened) if you disagree. I'm just tidying up and not acting on a super-strong opinion or anything like that.

@Trott Trott closed this as completed Jul 15, 2017
@Trott Trott removed the good first issue Issues that are suitable for first-time contributors. label Jul 15, 2017
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

No branches or pull requests

7 participants