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

tools,meta: remove exclusions from AUTHORS #40648

Closed
wants to merge 1 commit into from
Closed

tools,meta: remove exclusions from AUTHORS #40648

wants to merge 1 commit into from

Commits on Oct 29, 2021

  1. tools,meta: remove exclusions from AUTHORS 

    The update-authors tool omits Chromium team members. This was a decision
made years ago. The idea was that these folks contributed to V8 not to
Node.js, as they typically ended up in our code-base via cherry-picked
commits from V8.
Ref: nodejs#232 (diff)

Now that there are over 3000 contributors identified in AUTHORS, some of
whom are bots that we run, I think it's time to undo this exception. If
we used someone else's code in a cherry-pick, congratulations to them,
they're authors. In particular, this will fix the omission of Yang
Guo who contributed extensively to Node.js outside of V8 contributions.
    @Trott
    Trott committed Oct 29, 2021
    Configuration menu
    Copy the full SHA
    8f0d8d4 View commit details
    Browse the repository at this point in the history