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

test: remove setMaxListeners in test-crypto-random #35079

Merged
merged 1 commit into from
Sep 9, 2020

Conversation

tniessen
Copy link
Member

@tniessen tniessen commented Sep 6, 2020

This appears to be a remnant from 413d38c. Since common.mustCall never registers more than one listener, it is unnecessary.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@tniessen tniessen added crypto Issues and PRs related to the crypto subsystem. events Issues and PRs related to the events subsystem / EventEmitter. test Issues and PRs related to the tests. request-ci Add this label to start a Jenkins CI on a PR. labels Sep 6, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 6, 2020
@nodejs-github-bot
Copy link
Collaborator

This appears to be a remnant from
413d38c.

PR-URL: nodejs#35079
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@Trott Trott force-pushed the test-remove-setmaxlisteners branch from ccd36da to f07f420 Compare September 9, 2020 15:16
@Trott
Copy link
Member

Trott commented Sep 9, 2020

Landed in f07f420

@Trott Trott merged commit f07f420 into nodejs:master Sep 9, 2020
ruyadorno pushed a commit that referenced this pull request Sep 17, 2020
This appears to be a remnant from
413d38c.

PR-URL: #35079
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@ruyadorno ruyadorno mentioned this pull request Sep 21, 2020
4 tasks
addaleax pushed a commit that referenced this pull request Sep 22, 2020
This appears to be a remnant from
413d38c.

PR-URL: #35079
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@codebytere codebytere mentioned this pull request Sep 28, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
This appears to be a remnant from
413d38c.

PR-URL: nodejs#35079
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crypto Issues and PRs related to the crypto subsystem. events Issues and PRs related to the events subsystem / EventEmitter. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants