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

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

Closed
wants to merge 1 commit into from

Conversation

lpinca
Copy link
Member

@lpinca lpinca commented Oct 7, 2016

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
Copy link
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

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

LGTM

@MylesBorins
Copy link
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
Copy link
Member Author

lpinca commented Oct 7, 2016

@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
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