Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Allow for regular contributors to get away with 'infractions' #125

Closed
orta opened this issue Jul 7, 2020 · 3 comments
Closed

Allow for regular contributors to get away with 'infractions' #125

orta opened this issue Jul 7, 2020 · 3 comments

Comments

@orta
Copy link
Contributor

orta commented Jul 7, 2020

For modules which aren't too popular, and for users who have submitted a lot of PRs (like 20-30+) we could

  • Allow with no tests
  • Allow other reviewers
  • Allow PRs which touch tsconfig or tslint for that project

We could say only allow 1 of these at a time per PR, and to give at least a day before self merging is allowed.

@orta
Copy link
Contributor Author

orta commented Jul 7, 2020

@elibarzilay
Copy link
Contributor

I went over all of these, and I don't think that there's anything else needed beyond adding code to ignore config file changes that are OK.

@orta
Copy link
Contributor Author

orta commented Jul 22, 2020

Yeah, we don't need this anymore

@orta orta closed this as completed Jul 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants