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

doc: mention moderation repo in onboarding doc #10869

Closed
wants to merge 4 commits into from

Conversation

addaleax
Copy link
Member

What it says in the PR title. ;)

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. lts-watch-v6.x labels Jan 18, 2017
@addaleax addaleax added the meta Issues and PRs related to the general management of the project. label Jan 18, 2017
@@ -71,6 +71,9 @@ onboarding session.
* [**See "Who to CC in issues"**](./onboarding-extras.md#who-to-cc-in-issues)
* will also come more naturally over time

* When a discussion gets heated, you can request that other collaborators keep an eye on it by opening an issue at the [nodejs/moderation](https://github.com/nodejs/moderation) repository
Copy link
Member

Choose a reason for hiding this comment

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

Super tiny nit: can you add a final dot?

Copy link
Member Author

Choose a reason for hiding this comment

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

@lpinca done. ;)

@jasnell
Copy link
Member

jasnell commented Jan 18, 2017

Might be worthwhile mentioning that the Moderation Repo details are confidential and should not be shared externally

@addaleax
Copy link
Member Author

@jasnell done!

@jasnell
Copy link
Member

jasnell commented Jan 18, 2017

I don't believe this one needs to wait the full 48 hours to land

@addaleax
Copy link
Member Author

I don't believe this one needs to wait the full 48 hours to land

I’d prefer to wait, tbh… maybe other people have valid points like yours. We have one onboarding scheduled for today and @Trott can probably bring it up “manually” ;)

@Trott
Copy link
Member

Trott commented Jan 18, 2017

I prefer that the wording make it more clear that the people who have access to the repository number hundreds and are not merely the collaborators team. The way it is now, someone might think it's more private than it really is.

@addaleax
Copy link
Member Author

@Trott Updated, how does this wording work for you? I can include an actual number if you’d like to be even more explicit about it.

@Trott
Copy link
Member

Trott commented Jan 18, 2017

@addaleax That looks good to me. Maybe this might be even a bit better?:

This is a repository to which all members of the nodejs GitHub organization (not just Collaborators on Node.js core) have access.

@addaleax
Copy link
Member Author

@Trott your phrasing works for me, too … updated :)

@addaleax
Copy link
Member Author

Landed in 0c58193

@addaleax addaleax closed this Jan 21, 2017
@addaleax addaleax deleted the doc-onboarding-moderation branch January 21, 2017 07:58
addaleax added a commit that referenced this pull request Jan 21, 2017
PR-URL: #10869
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 24, 2017
PR-URL: nodejs#10869
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 27, 2017
PR-URL: nodejs#10869
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
@italoacasas italoacasas mentioned this pull request Jan 29, 2017
MylesBorins pushed a commit that referenced this pull request Mar 7, 2017
PR-URL: #10869
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
MylesBorins pushed a commit that referenced this pull request Mar 9, 2017
PR-URL: #10869
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
@MylesBorins MylesBorins mentioned this pull request Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants