Artur Gajowy: We should make sure our work is broken up into reviewable PR's. Tickets should take ~ 1-3 days. PR's should happen every ~ 1-3.
Ovidiu Deac: Don't perform refactoring + net new logic work in the same pull request. If you need refactoring as part of your implementation, cut a PR for your refactoring/reformatting first and separately - check for clean tests. Makes it easier. Then create a PR for the new features.
Artur Gajowy: Whenever you are changing repetitive code (modify it in different places) - step back and extract the duplicated function first. The worst thing that can happen is that you have improved the code quality. The best thing to happen is that you have reduced the size of the Pull Request.
Type your task here, using "@" to assign to a user and "//" to select a due date