Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

Add dshaw and ljharb as NodeSlackers liasons - #410

Merged
refack merged 4 commits into
nodejs:masterfrom
chasingSublimity:master
Nov 4, 2018
Merged

Add dshaw and ljharb as NodeSlackers liasons#410
refack merged 4 commits into
nodejs:masterfrom
chasingSublimity:master

Conversation

@chasingSublimity

Copy link
Copy Markdown
Contributor

Fixes #381

Is it worth adding some copy stating that collaborator liaisons are the point of contact for their given communities? Granted, the term liaison might make any clarification redundant, but I personally wouldn't think to reach out to the listed Node collaborators if I had trouble getting an invitation to a community.

The copy could be as simple as

Any questions regarding joining or participating in a partner community should be directed to the Node.js Collaborator liaison.

@mhdawson mhdawson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@refack

refack commented Nov 2, 2018

Copy link
Copy Markdown
Contributor

Hello @chasingSublimity and thank you for the contribution 🥇

The copy could be as simple as

Any questions regarding joining or participating in a partner community should be directed to the Node.js Collaborator liaison.

+1 from me.

@refack refack self-assigned this Nov 2, 2018

@bnb bnb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Love the clarification 💚

Comment thread PARTNER_COMMUNITIES.md Outdated
@refack
refack merged commit 32372ec into nodejs:master Nov 4, 2018
@refack

refack commented Nov 4, 2018

Copy link
Copy Markdown
Contributor

Congratulations @chasingSublimity on getting promoted by GitHub from:
image
to
image

Hope to see you contributing again in the future 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants