Когда удаляешь блок кода, который вроде бы не используется⁠⁠

Это не пантера с землей упала, это ветка вверх улетела:)) А если серьезно, то тупо набрать в поиске по проекту имя метода можно же, если уж совсем по детскому:))

use с английского еще переводится как "польза" и именно это значение используется в useless.

Атомарность коммитов. Если в задаче были затронуты 5 классов, два конфига и 11 тестов, то слить это все в один можно, но потом ни урл старый в конфиге не вернешь через git revert, не изменения в одном классе со связанными тестами и т. д.

Например, веб-сервисы с 4-6 абстракциями выжирают у меня порядка 35 коммитов минимум. Если новое, то каждый класс (можно с тестами) отдельно, правки настроек отдельно, обновление модулей отдельно и т. д. Была задача на бизнес-процесс, который включил 6 сервисов. Один был жестяк по объему логики. На все ушло где-то 1200 коммитов и более тысячи юнит и интеграционных тестов.

Почему плохой тон? Коммить сколько влезет, потом git squash как закончил

Пропел.

useless это "бесполезный", а не "не используемый".