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

cluster: remove deprecated problematic API #13684

Closed
wants to merge 1 commit into from

Commits on Jun 14, 2017

  1. cluster: remove deprecated problematic API

    Content Warning: mental health and self-harm triggers.
    
    ---
    
    The word "suicide" was used in the cluster API 4 years ago.  This was a
    serious mistake for several reasons.  It was deprecated in version 6,
    and long past time for full removal now.
    
    First, and most important, casual use of words referring to self-harm
    serve to stigmatize mental illness and make it harder for people in our
    community to literally stay alive.  It is impossible to claim that we
    intend to be inclusive and welcoming, and also casually show such
    disrespect for the lives of those in our community.  Some of us have
    lost friends or struggled with mental illness.  Leaving this lying
    around tells us that we are not welcome.
    
    Second, in addition to being just wildly inappropriate on ethical and
    community grounds, the term "suicide" was never even appropriate
    technically!  In context, it refers to a case where a child process
    exits without receiving a fatal signal from the parent process.  The
    "cute" unix joke here is that the process "killed itself".  However, the
    "suicide" flag is set regardless of how the process exited, so in
    addition to being offensive and harmful, it is also misleading and
    incorrect, and has always been so.  (It could've thrown an error, called
    `process.exit()` or received a fatal signal from some other process.)
    
    An open source project is not the place for emotional landmines.  It's
    distracting and misleading at best, and actively harmful at worst.
    There is no justification for it.  The API has already been renamed, and
    it's an obscure surface that is unlikely to ever be used outside of
    core.
    isaacs committed Jun 14, 2017
    Configuration menu
    Copy the full SHA
    08ebbb3 View commit details
    Browse the repository at this point in the history