nit: this could be a const
Мелочь: здесь можно const. nit — придирка, необязательная правка.
nit — мелкое замечание
Could we add a test for the empty case?
Можем добавить тест на пустой случай?. Could we … — мягкая просьба; we — «мы вместе».
✗ Add test for empty case.
empty case — пустой случай
I'm not sure this handles null. What do you think?
Не уверен, что тут обрабатывается null. Как думаешь?. Сомнение и вопрос вместо утверждения «это неправильно».
✗ This is wrong with null.
handles — обрабатывает
Suggestion: extract this into a helper.
Предложение: вынести это в функцию.. Suggestion: — явно помечает, что это совет.
suggestion — предложение
Can you rebase onto main?
Можешь сделать rebase на main?. rebase onto — переставить ветку поверх.
✗ Can you rebase to main?
onto — на (поверх)
LGTM, thanks for the quick fix!
Выглядит хорошо, спасибо за быстрый фикс!. LGTM — looks good to me, одобрение.
LGTM — выглядит хорошо
Good catch! Fixed in 3f2a9c1.
Хорошо замечено! Исправлено в 3f2a9c1.. Good catch — ответ на найденную ошибку.
good catch — хорошо подмечено