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

Suggest number of reviewers based on change complexity #31

Closed ikbalkaya closed 2 years ago

ikbalkaya commented 2 years ago

As a follow up to https://github.com/ably/engineering/issues/29

I added some suggestions on which instances to seek one, two or more reviewers.

Closes https://github.com/ably/engineering/issues/29

QuintinWillison commented 2 years ago

@stmoreau @AndyNicks @ikbalkaya I've made some tweaks. Are they ok?

ikbalkaya commented 2 years ago

@stmoreau @AndyNicks @ikbalkaya I've made some tweaks. Are they ok?

@QuintinWillison I realised a typo and made a new commit but otherwise Ok from my side.