Skip to content

lib: consolidate redundant require() calls#21699

Merged
cjihrig merged 1 commit intonodejs:masterfrom
cjihrig:no-dup-reqs
Jul 10, 2018
Merged

lib: consolidate redundant require() calls#21699
cjihrig merged 1 commit intonodejs:masterfrom
cjihrig:no-dup-reqs

Conversation

@cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Jul 7, 2018

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

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot added the lib / src Issues and PRs related to general changes in the lib or src directory. label Jul 7, 2018
@Trott
Copy link
Member

Trott commented Jul 7, 2018

Another spurious linter error that only seems to show up in auto-started linter jobs.

PR-URL: nodejs#21699
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Jon Moss <[email protected]>
@cjihrig
Copy link
Contributor Author

cjihrig commented Jul 10, 2018

CI with one flake that failed: https://ci.nodejs.org/job/node-test-pull-request/15776/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

8 participants