Skip to content

Conversation

@Trott
Copy link
Member

@Trott Trott commented Feb 6, 2016

One function is at incorrect heading/indentation. This fixes it.

@Trott Trott added dgram Issues and PRs related to the dgram subsystem / UDP. doc Issues and PRs related to the documentations. lts-watch-v4.x labels Feb 6, 2016
@Trott
Copy link
Member Author

Trott commented Feb 6, 2016

/cc @nodejs/documentation

@stevemao
Copy link
Contributor

stevemao commented Feb 6, 2016

Line 376 missing a newline.

@Trott
Copy link
Member Author

Trott commented Feb 6, 2016

@stevemao Fixed and force pushed.

@stevemao
Copy link
Contributor

stevemao commented Feb 6, 2016

Thanks @Trott :)

@silverwind
Copy link
Contributor

LGTM

Trott added a commit to Trott/io.js that referenced this pull request Feb 8, 2016
@Trott
Copy link
Member Author

Trott commented Feb 8, 2016

Landed in 394556e

@Trott Trott closed this Feb 8, 2016
rvagg pushed a commit that referenced this pull request Feb 9, 2016
PR-URL: #5118
Reviewed-By: Roman Reiss <[email protected]>
rvagg pushed a commit that referenced this pull request Feb 9, 2016
PR-URL: #5118
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 18, 2016
PR-URL: #5118
Reviewed-By: Roman Reiss <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 18, 2016
PR-URL: #5118
Reviewed-By: Roman Reiss <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Feb 18, 2016
MylesBorins pushed a commit that referenced this pull request Mar 2, 2016
PR-URL: #5118
Reviewed-By: Roman Reiss <[email protected]>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
@Trott Trott deleted the dgram-indent branch January 13, 2022 22:42
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.

4 participants