doc: update list of personal traits in CoC by zkat · Pull Request #4801 · nodejs/node · GitHub

rust-lang/prev.rust-lang.org#288 landed in the Rust repo so it seems like a good idea to just bring the updated list in.

We also received a request to do this in nodejs/inclusivity#82 so this should resolve that.

This PR would preferably also be merged into LTS, imo.

/cc @nodejs/inclusivity @nodejs/lts @nodejs/documentation

cc @Charlotteis

@sup yes, that is perfectly valid in English.

I believe Rust adapted this from the Contributor Covenant so that might be worth including in the first sentence about adaptations?

fixed

is this supposed to say characteristic in singular?

I think it works how it is, yeah.

LGTM. This isn't actually a policy change, so there's no reason for this to go to a CTC meeting imo.

Thanks very much @zkat ✨

@Charlotteis @zkat ok thanks! wasn't sure

lgtm... I especially like the inclusion of level of experience

Very nice update. I'm glad the org is taking this direction.

LGTM

I agree with @Fishrock123 ... there shouldn't be a need to put this on the meeting agenda. I'd give it another day tho before landing.

Out of curiosity, why is there an explicit list of characteristics anyway, especially if it could just go on and on forever like this since it's possible for anyone to discriminate for any reason? Isn't "we are committed to providing a friendly, safe and welcoming environment for all" enough?

LGTM

@mscdex well presumably, if someone feels excluded from participating because of one of those traits from the list - they might feel better if they know they are explicitly encouraged to participate.

Or something like that.

if someone feels excluded from participating because of one of those traits from the list - they might feel better if they know they are explicitly encouraged to participate.

This is largely why I proposed this in the first place, yeah 😄

@mscdex Explicit is better than implicit. One might try to argue that the lack of inclusion of a particular form of bigotry in the list is validation that it's "okay" when it really is not.

Updates still LGTM

:lgtm: ✨

hey @jasnell do we think we'll merge today or next week? just curious.

really proud of our community's response on this thread. go node ❤️ 😊

@ashleygwilliams I don't see why we wouldn't merge it today. Since @zkat is a collaborator we defer to her to do the actual merge unless otherwise noted. :)

rust-lang/prev.rust-lang.org#288 landed in the Rust repo so it seems like a good idea to just bring the updated list in. We also received a request to do this in nodejs/inclusivity#82 so this should resolve that. Thanks to [@Charlotteis](https://github.com/Charlotteis) for bringing up the original issue. Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Myles Borins <mborins@us.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> PR-URL: nodejs#4801 Fixes: nodejs/inclusivity#82

rust-lang/prev.rust-lang.org#288 landed in the Rust repo so it seems like a good idea to just bring the updated list in. We also received a request to do this in nodejs/inclusivity#82 so this should resolve that. Thanks to [@Charlotteis](https://github.com/Charlotteis) for bringing up the original issue. Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Myles Borins <mborins@us.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> PR-URL: #4801 Fixes: nodejs/inclusivity#82

Landed in master as of de0d35f 🎉

💖

👍

Suggestion: Change religion to religion or lack thereof.

@chriseppstein patches welcome 😁

FWIW, we just added that language to the inclusivity CoC: nodejs/inclusivity#97. Thanks for the suggestion @chriseppstein!

Closing because it landed :-) We'll definitely get this pulled back into the LTS... hopefully in the next round

Would or any other characteristic have been better?

@zkat Can you keep lines in the commit log <= 72 columns next time? Thanks.

rust-lang/prev.rust-lang.org#288 landed in the Rust repo so it seems like a good idea to just bring the updated list in. We also received a request to do this in nodejs/inclusivity#82 so this should resolve that. Thanks to [@Charlotteis](https://github.com/Charlotteis) for bringing up the original issue. Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Myles Borins <mborins@us.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> PR-URL: #4801 Fixes: nodejs/inclusivity#82

rust-lang/prev.rust-lang.org#288 landed in the Rust repo so it seems like a good idea to just bring the updated list in. We also received a request to do this in nodejs/inclusivity#82 so this should resolve that. Thanks to [@Charlotteis](https://github.com/Charlotteis) for bringing up the original issue. Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Myles Borins <mborins@us.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> PR-URL: nodejs#4801 Fixes: nodejs/inclusivity#82

zkat added the doc Issues and PRs related to the documentations. label Jan 21, 2016

varjmes reviewed Jan 21, 2016
View reviewed changes

Fishrock123 added the meta Issues and PRs related to the general management of the project. label Jan 21, 2016

zkat force-pushed the gender-identity-patch branch from c17c6f3 to f31443b Compare January 21, 2016 19:26

zkat changed the title doc: update list of personal trais in CoC doc: update list of personal traits in CoC Jan 21, 2016

zkat force-pushed the gender-identity-patch branch from f31443b to d293d02 Compare January 21, 2016 19:30

zkat force-pushed the gender-identity-patch branch from d293d02 to 974331d Compare January 21, 2016 20:38

jasnell added the lts-watch-v4.x label Jan 21, 2016

zkat force-pushed the gender-identity-patch branch from 974331d to de0d35f Compare January 22, 2016 21:34

jasnell closed this Jan 22, 2016

thefourtheye reviewed Jan 23, 2016
View reviewed changes

MylesBorins added land-on-v4.x and removed lts-watch-v4.x labels Jan 28, 2016

rvagg mentioned this pull request Feb 8, 2016

Release proposal: v5.5.1 (Stable) #5141

Closed

MylesBorins mentioned this pull request Feb 11, 2016

V4.3.1 proposal #5200

Merged

doc: update list of personal traits in CoC by zkat · Pull Request #4801 · nodejs/node · GitHub

Uh oh!

Uh oh!

doc: update list of personal traits in CoC - #4801

doc: update list of personal traits in CoC#4801zkat wants to merge 1 commit intonodejs:masternodejs/node:masterfrom zkat:gender-identity-patchzkat/node:gender-identity-patchCopy head branch name to clipboard

Conversation

zkat commented Jan 21, 2016

Uh oh!

ghost commented Jan 21, 2016

Uh oh!

zkat commented Jan 21, 2016

Uh oh!

varjmes Jan 21, 2016

Choose a reason for hiding this comment

Uh oh!

zkat Jan 21, 2016

Choose a reason for hiding this comment

Uh oh!

varjmes commented Jan 21, 2016

Uh oh!

Fishrock123 commented Jan 21, 2016

Uh oh!

varjmes commented Jan 21, 2016

Uh oh!

ghost commented Jan 21, 2016

Uh oh!

MylesBorins commented Jan 21, 2016

Uh oh!

benjamingr commented Jan 21, 2016

Uh oh!

jasnell commented Jan 21, 2016

Uh oh!

jasnell commented Jan 21, 2016

Uh oh!

mscdex commented Jan 21, 2016

Uh oh!

ChALkeR commented Jan 21, 2016

Uh oh!

benjamingr commented Jan 21, 2016

Uh oh!

varjmes commented Jan 21, 2016

Uh oh!

Qard commented Jan 21, 2016

Uh oh!

Fishrock123 commented Jan 22, 2016

Uh oh!

juliepagano commented Jan 22, 2016

Uh oh!

ashleygwilliams commented Jan 22, 2016

Uh oh!

Fishrock123 commented Jan 22, 2016

Uh oh!

zkat commented Jan 22, 2016

Uh oh!

ashleygwilliams commented Jan 22, 2016

Uh oh!

msikma commented Jan 22, 2016

Uh oh!

chriseppstein commented Jan 22, 2016

Uh oh!

zkat commented Jan 22, 2016

Uh oh!

nebrius commented Jan 22, 2016

Uh oh!

jasnell commented Jan 22, 2016

Uh oh!

thefourtheye Jan 23, 2016

Choose a reason for hiding this comment

Uh oh!

bnoordhuis commented Jan 23, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

16 participants

doc: update list of personal traits in CoC#4801
zkat wants to merge 1 commit into
nodejs:masterfrom
zkat:gender-identity-patch