Skip to content

doc: reference list of language-specific globals - #6900

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:doc-language-globals
Closed

doc: reference list of language-specific globals#6900
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:doc-language-globals

Conversation

@addaleax

Copy link
Copy Markdown
Member
Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Fixes: #6894

@addaleaxaddaleax added the doc Issues and PRs related to the documentations. label May 20, 2016
Comment threaddoc/api/globals.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the second sentence should start with "There are a number...", but I'm not sure. Feel free to ignore.

@cjihrig

Copy link
Copy Markdown
Contributor

LGTM

@addaleax
addaleaxforce-pushed the doc-language-globals branch from d531f54 to fc4ab82CompareMay 20, 2016 18:09
@addaleax

Copy link
Copy Markdown
MemberAuthor

Updated anyway, both sound okay to me but you’re the native speaker here. :)

@jasnell

Copy link
Copy Markdown
Member

LGTM

1 similar comment
@eljefedelrodeodeljefe

Copy link
Copy Markdown
Contributor

LGTM

@addaleax

Copy link
Copy Markdown
MemberAuthor

Landed in 2193a2e

@addaleax
addaleax deleted the doc-language-globals branch May 23, 2016 23:22
addaleax added a commit that referenced this pull request May 23, 2016
Fixes: #6894
PR-URL: #6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request May 30, 2016
Fixes: nodejs#6894
PR-URL: nodejs#6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
rvagg pushed a commit that referenced this pull request Jun 2, 2016
Fixes: #6894
PR-URL: #6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jun 3, 2016
Fixes: #6894
PR-URL: #6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jun 24, 2016
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Fixes: #6894
PR-URL: #6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Fixes: #6894
PR-URL: #6900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Robert Jefe Lindstaedt <robert.lindstaedt@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@addaleax@cjihrig@jasnell@eljefedelrodeodeljefe@MylesBorins