Skip to content

Conversation

@Trott
Copy link
Member

@Trott Trott commented Oct 23, 2017

ping @nodejs/tsc @nodejs/moderation

Copy link
Contributor

@MylesBorins MylesBorins left a comment

Choose a reason for hiding this comment

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

LGTM

@MylesBorins
Copy link
Contributor

@Trott we may want to do this in the admin repo

The Moderation doc is living in two places. We likely should move this doc over there first

@vsemozhetbyt
Copy link

@othiym23
Copy link

@MylesBorins It's already there (I only figured that out on Friday myself, don't feel bad). @Trott -- you want to close this and move the proposal there? And should we start the process of removing the moderation policy here? It's at best confusing having it in two places, even if one of them is a deprecated repository.

@othiym23
Copy link

Also, on the proposal's merits: I think this is a sensible proposal, and also would be interested in basically codifying a "quick to ban, quick to forgive" policy around bots or any other behavior that can rapidly trigger events in repos, because it's so easy for bot-like activity to create a lot of work for a lot of people.

@MylesBorins
Copy link
Contributor

I've opened #395

Copy link
Member

@benjamingr benjamingr left a comment

Choose a reason for hiding this comment

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

LGTMing to show agreement.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@MylesBorins
Copy link
Contributor

This is going to have to be moved to the admin repo

@Trott
Copy link
Member Author

Trott commented Nov 26, 2017

This is going to have to be moved to the admin repo.

nodejs/admin#31

@Trott Trott closed this Nov 26, 2017
@Trott Trott deleted the no-bots-but-our-bots branch March 11, 2021 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.