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.

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

fix: correct removeAllListeners in case no event is passed #2088

Merged
merged 4 commits into from
Apr 9, 2021
Merged

fix: correct removeAllListeners in case no event is passed #2088

merged 4 commits into from
Apr 9, 2021

Commits on Apr 8, 2021

  1. fix: correct removeAllListeners in case no event is passed 

    Correct handling of patched removeAllListeners() in AbstractAsyncHooksContextManager in case
it's called without argument.

In this case all listener types should be removed.
Additionally it's imporant to forward arguments instead of passing undefined as node.js checks arguments.length.
    @Flarna
    Flarna committed Apr 8, 2021
    Configuration menu
    Copy the full SHA
    057e9d9 View commit details
    Browse the repository at this point in the history

  2. fixup! use strict equal and Object.create(null)

    @Flarna
    Flarna committed Apr 8, 2021
    Configuration menu
    Copy the full SHA
    ab8683a View commit details
    Browse the repository at this point in the history

Commits on Apr 9, 2021

  1. Merge branch 'main' into remove-all-listeners

    @Flarna
    Flarna authored Apr 9, 2021
    Configuration menu
    Copy the full SHA
    df0bc39 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into remove-all-listeners

    @vmarchaud
    vmarchaud authored Apr 9, 2021
    Configuration menu
    Copy the full SHA
    dd9e1bf View commit details
    Browse the repository at this point in the history