Skip to content

Conversation

lpinca
Copy link
Member

@lpinca lpinca commented Aug 20, 2016

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

doc

Description of change

Ref: #6578

@nodejs-github-bot nodejs-github-bot added dgram Issues and PRs related to the dgram subsystem / UDP. doc Issues and PRs related to the documentations. labels Aug 20, 2016

### socket.bind([port][, address][, callback])
<!-- YAML
added: v0.1.99
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The callback argument was actually added in version 0.9.1. I'm not sure what version to use.
There are other inconsistencies like this below.

Copy link
Contributor

@yorkie yorkie Aug 20, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be noted as updated: v0.9.1 even thought we have not that option :-(

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC, we have not documenting when an argument or something like a property in an options object is added.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, we don't currently document changes well. It'd be nice if we had per-function changelogs, like PHP.

@Trott
Copy link
Member

Trott commented Aug 23, 2016

bump

@Trott
Copy link
Member

Trott commented Aug 23, 2016

LGTM although I haven't checked the validity of the data. Not sure if that's supposed to be part of the process on these or not.

@jasnell
Copy link
Member

jasnell commented Aug 24, 2016

@Trott ... yes, double checking the validity of the versions is a good thing but it's also something that can be fixed separately if some of the details are off.

@jasnell
Copy link
Member

jasnell commented Aug 24, 2016

LGTM

jasnell pushed a commit that referenced this pull request Aug 25, 2016
PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@jasnell
Copy link
Member

jasnell commented Aug 25, 2016

Landed in 379d916

@jasnell jasnell closed this Aug 25, 2016
@lpinca lpinca deleted the add/added-dgram branch August 25, 2016 18:31
@Fishrock123 Fishrock123 mentioned this pull request Sep 6, 2016
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request Sep 8, 2016
PR-URL: nodejs#8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Fishrock123 pushed a commit that referenced this pull request Sep 9, 2016
PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@MylesBorins
Copy link
Contributor

this is not landing cleanly on v4.x please feel free to backport

lpinca added a commit to lpinca/node that referenced this pull request Oct 1, 2016
PR-URL: nodejs#8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 4, 2016
Ref: #8883

PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 10, 2016
Ref: #8883

PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
Ref: #8883

PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
Ref: #8883

PR-URL: #8196
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dgram Issues and PRs related to the dgram subsystem / UDP. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants