search

ably / engineering

Ably's new home for engineering guidance, including content previously served from Confluence at engineering.ably.com.
https://engineering.ably.com/
Apache License 2.0
2 stars 6 forks source link

Make a distinction explanation for PR reviewer counts for trivial / non-trivial changes. #29

Closed ikbalkaya closed 2 years ago

ikbalkaya commented 2 years ago

Even if the enforced number of reviewers are only one, we need to provide some explanation where this is not a good idea. For example for some non-trivial changes there should be at least 2 reviewers. For some even more. Explain this in a relevant section.