FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

src: name all builtin init functions Initialize by danbev · Pull Request #19550 · nodejs/node · GitHub

/ node Public

src: name all builtin init functions Initialize - #19550

Closed
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:builtin_module_initialize
Closed

src: name all builtin init functions Initialize#19550
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:builtin_module_initialize

Conversation

danbev commented Mar 23, 2018

Copy link
Copy Markdown
Contributor

This commit renames a few of the builtin modules init functions to
Initialize for consistency.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

This commit renames a few of the builtin modules init functions to
Initialize for consistency.
nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. labels Mar 23, 2018

danbev commented Mar 23, 2018

Copy link
Copy Markdown
Contributor Author

danbev commented Mar 23, 2018

Copy link
Copy Markdown
Contributor Author
node-test-commit-arm-fanned failure looks unrelated

console output:

07:10:23 not ok 207 parallel/test-stdio-pipe-redirect
07:10:23   ---
07:10:23   duration_ms: 7.871
07:10:23   severity: fail
07:10:23   stack: |-
07:10:23     pure virtual method called
07:10:23     terminate called without an active exception

danbev commented Mar 26, 2018

Copy link
Copy Markdown
Contributor Author

Landed in ebbf393.

danbev closed this Mar 26, 2018
danbev deleted the builtin_module_initialize branch March 26, 2018 06:32
danbev added a commit that referenced this pull request Mar 26, 2018
This commit renames a few of the builtin modules init functions to
Initialize for consistency.

PR-URL: #19550
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Apr 2, 2018
This commit renames a few of the builtin modules init functions to
Initialize for consistency.

PR-URL: #19550
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos mentioned this pull request Apr 4, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL