Зачем нужен вежливый тон на Code Review?
Часто разработчики пишут прямолинейно: «This is wrong, fix it». В интернациональной команде такой тон может быть воспринят как агрессия.
1. Как попросить изменить архитектуру
- ❌ You should rewrite this part.
- ✅ Have you considered using a different pattern here? It might scale better.
- ✅ I wonder if extracting this into a separate class would make it more testable.
2. Указание на ошибку в логике
- ❌ This will crash if data is null.
- ✅ It looks like this might throw an error if the data is null. Could we add a check?
- ✅ Just a heads up, this corner case isn't handled yet.
Помните, наша цель — улучшить код, а не обидеть коллегу. Используйте эти простые смягчения в своей ежедневной работе с pull-реквестами.