Добавляет новые правила под новые критерии

polyuretanio commented 3 years ago

[x] От формулировок "Правильно/неправильно" лучше вернуться к формулировкам "Хорошо/плохо"
[x] В автоматизации проверок кода надо сказать, что это не ссылки на сами инструменты, а ссылки на конфигурации аккадемии для этих инструментов. И эти конфигурации академии частично покрывают то, что описано в кодгаде ниже. И в конфигурациях нет ничего, что не описано ниже.
[x] Кодгайд более рекомендательная штука, поэтому можно использовать более мягкую модальность. То есть уйти от "должны". Например, вместо "Файлы схожих типов должны быть сгруппированы в папки: картинки, стили, скрипты, шрифты." можно написать "Файлы схожих типов группируются в папки: картинки, стили, скрипты, шрифты."
[x] странный пункт про "Форматирование кода" и эдитор конфиг. Кодгайд про конкретику, а в этом пункте нет никакой конкретики
[x] Форматирование тегов здесь у меня первое расхождение с кодгайдом нашим, я не всегда переношу каждый тег на новую строку и считаю, что это вполне ок
[x] Есть ощущение, что многие отдельные пункты новой версии можно сгруппировать в списки для большей компактности, как в старом кодгайде
[x] "Единицы измерения в атрибутах" а если есть такие атрибуты, в которых единицы измерения это ок? Да и вообще это разве пункт кодгайда? Это же просто спекой запрещено
[x] "Подписи полей ввода" это дубль критерия, зачем он в кодгайде?
[x] "Подключение шрифтов" тоже дубль критерия и непонятно зачем это в кодгайде
[x] "Варианты шрифта" надо обновить в сооветствии с критерием, и это тоже дубль

Резюмируя: 99% вышеперечисленного это комментарии по форме представления кодгайда, а не по его содержанию

nikolai-shabalin commented 3 years ago

@polyuretanio

странный пункт про "Форматирование кода" и эдитор конфиг. Кодгайд про конкретику, а в этом пункте нет никакой конкретики

А чего не хватает? Вроде всё явно. Следуй эдиторкофигу. Прямое требование

polyuretanio commented 3 years ago

@polyuretanio

странный пункт про "Форматирование кода" и эдитор конфиг. Кодгайд про конкретику, а в этом пункте нет никакой конкретики

А чего не хватает? Вроде всё явно. Следуй эдиторкофигу. Прямое требование

Просто это не выглядит куском кодгайда. Должен быть пример конкретного конфига. А не просто "используйте такой-то инструмен". Точно также можно попросить "использовать препроцесор" или "использовать методологию названия классов", причем без уточнения, что за препроцессор или методология.

nikolai-shabalin commented 3 years ago

Угу, понял

htmlacademy / codeguide

Добавляет новые правила под новые критерии #52