Skip to content

util: named anonymous functions #20408

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

util: named anonymous functions #20408

wants to merge 2 commits into from

Conversation

ccoxwell
Copy link
Contributor

named two anonymous functions in util.js. references #8913.

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

@nodejs-github-bot nodejs-github-bot added the util Issues and PRs related to the built-in util module. label Apr 29, 2018
@starkwang
Copy link
Contributor

CI: https://ci.nodejs.org/job/node-test-pull-request/14595/

@mmarchini
Copy link
Contributor

Landed in a00f76c...0930d7e. Thanks and congratulations on your first contribution! 🎉🎉🎉🎉🎉

@mmarchini mmarchini closed this May 3, 2018
mmarchini pushed a commit that referenced this pull request May 3, 2018
@ccoxwell
util: named anonymous functions
4d8806f 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
mmarchini pushed a commit that referenced this pull request May 3, 2018
@ccoxwell
lib: named anonymous functions
0930d7e 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 4, 2018
@ccoxwell @MylesBorins
util: named anonymous functions
43ba7a6 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 4, 2018
@ccoxwell @MylesBorins
lib: named anonymous functions
81584d1 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
@MylesBorins MylesBorins mentioned this pull request May 8, 2018
Merged
MylesBorins pushed a commit that referenced this pull request May 8, 2018
@ccoxwell @MylesBorins
util: named anonymous functions
1b9c40c 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 8, 2018
@ccoxwell @MylesBorins
lib: named anonymous functions
b6de6a7 
PR-URL: #20408
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lpinca lpinca lpinca approved these changes

@targos targos targos approved these changes

@cjihrig cjihrig cjihrig approved these changes

+2 more reviewers

@mmarchini mmarchini mmarchini approved these changes

@starkwang starkwang starkwang approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

util Issues and PRs related to the built-in util module.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@ccoxwell @starkwang @mmarchini @lpinca @targos @cjihrig @nodejs-github-bot