Що ви перевіряєте під час code review?

Що зміна робить заявлене і не ламає суміжне; читабельність і відповідність стандартам (WPCS); безпеку — sanitizing, escaping, nonce, перевірку прав; продуктивність запитів; наявність тестів. Коментарі пишу конкретно і з пропозицією, дрібниці позначаю як необов'язкові.

Коли варто використовувати squash merge?

Коли проміжні коміти гілки («fix typo», «wip») не мають цінності для історії: squash зводить PR в один змістовний коміт у main. Не варто, коли PR містить кілька окремих логічних змін, які корисно бачити й відкочувати окремо.