mainmatter / playbook

A book describing the patterns and practices that Mainmatter uses to build lasting products, systematically.
https://simplabs.com/playbook/
13 stars 1 forks source link

add more clarification to resolver role on reviews #134

Closed izelnakri closed 3 years ago

izelnakri commented 3 years ago

Additional paragraph added to Reviews section to clarify the role, the need and responsibilities of the resolver more. I see that there is an avoidance to elect or take the role of the resolver in our team effectively, so this paragraph is needed. Of course calling a resolver is the last option but we shouldn't prevent people from calling a resolver if people get blocked for various reasons, so I think this paragraph is very useful and encourages people to call a resolver since people won't doubt whether its a good idea, a better clarification is made.

Tensions/conflicts when multiple people work together is inevitable, so it is a good idea to stress that and explain the path on how to resolve tensions/conflicts/disagreements.

marcoow commented 3 years ago

No description provided.

https://github.com/simplabs/playbook/tree/master/src/development-process#feature-branches-and-pull-requests 🙏

izelnakri commented 3 years ago

No description provided.

https://github.com/simplabs/playbook/tree/master/src/development-process#feature-branches-and-pull-requests

added a description.