Skip to content

v4.x backport: doc: add added: information for globals#8964

Closed
lpinca wants to merge 1 commit into
nodejs:v4.x-stagingfrom
lpinca:backport/8901
Closed

v4.x backport: doc: add added: information for globals#8964
lpinca wants to merge 1 commit into
nodejs:v4.x-stagingfrom
lpinca:backport/8901

Conversation

@lpinca

@lpinca lpinca commented Oct 7, 2016

Copy link
Copy Markdown
Member

v4.x backport of #8901

cc: @thealphanerd

Ref: nodejs#6578
PR-URL: nodejs#8901
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. globals labels Oct 7, 2016

@jasnell jasnell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@MylesBorins

Copy link
Copy Markdown
Contributor

landed in 055f5c69ab

@lpinca you will notice that I swapped what you had put for PR-URL and Ref.

I have generally been keeping the original PR-URL for backports, mostly due to how our tooling works.

@MylesBorins MylesBorins closed this Oct 7, 2016
@lpinca

lpinca commented Oct 7, 2016

Copy link
Copy Markdown
Member Author

@thealphanerd I'm a bit confused. Now PR-URL points to an issue.

@lpinca lpinca deleted the backport/8901 branch October 7, 2016 17:00
MylesBorins pushed a commit that referenced this pull request Oct 7, 2016
Ref: #6578
Ref: #8964
PR-URL: #8901
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@MylesBorins

Copy link
Copy Markdown
Contributor

oh doodle... I totally messed that up. Fixing and force pushing :D
ab08cc2

MylesBorins pushed a commit that referenced this pull request Oct 10, 2016
Ref: #6578
Ref: #8964
PR-URL: #8901
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
Ref: #6578
Ref: #8964
PR-URL: #8901
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
Ref: #6578
Ref: #8964
PR-URL: #8901
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
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

Successfully merging this pull request may close these issues.

4 participants