search

bebop / poly

A Go package for engineering organisms.
https://pkg.go.dev/github.com/bebop/poly
MIT License
663 stars 70 forks source link

Reduce number of reviewers necessary for merge #372

Closed carreter closed 8 months ago

carreter commented 10 months ago

At least 6 approving reviews are required by reviewers with write access

Maybe this should be 2?

Originally posted by @Koeng101 in https://github.com/TimothyStiles/poly/pull/358#issuecomment-1732223779

Agreed with @Koeng101 here. Currently @TimothyStiles has to approve all PRs. IMO we should set it to 1 for now and bump it to 2 once we add another person who's willing to consistently contribute to the team.

Koeng101 commented 10 months ago

@TimothyStiles !

carreter commented 10 months ago

Getting the GitHub org set up would also be helpful - having more people (specifically: me) be able to change repo settings would be nice!

carreter commented 8 months ago

Seems like the move to the @bebop org has raised this to 6 again...

TimothyStiles commented 8 months ago

Seems like the move to the @bebop org has raised this to 6 again...

@carreter working on it.

TimothyStiles commented 8 months ago

@carreter done