yomoyo icon indicating copy to clipboard operation
yomoyo copied to clipboard

[intensive][HTML2] Упразднить критерии сборки

Open firefoxic opened this issue 1 year ago • 3 comments

Ссылка

https://up.htmlacademy.ru/adaptive/30/criteries#BUILD

Проблема или вопрос

Раздел про сборку (восьмой) теперь факультативный. А критерии про сборку даже без ⭐️

Дополнительная информация

Студенты сами сборку не пишут, бо́льшая часть требований просто неактуальна.

Возможное решение или предложение по улучшению

Можно последовательно от них избавиться:

  1. Прямо сейчас можно проставить звёздочки, чтобы не только раздел, но и его критерии были тоже факультативными.
  2. Для следующего потока можно совсем удалить их.

Как вариант можно удалять не все, оставив «Команды запуска», при этом разделив его на два:

  1. критерий про npm run start, описывающий, что должно быть при запуске дев-сервера (больше про то, что вёрстка должна быть готовой и все её файлы лежат на своих местах);
  2. критерий про npm run build, описывающий, что при сборке в прод в папке build/ должно быть всё необходимое для работы сайта (можно тут же упомянуть npm run preview).

firefoxic avatar Dec 30 '23 08:12 firefoxic

Удалять критерии не будем, так как они защищают от модификаций сборки. Особенно со стороны наставников.

В остальном согласен. Звёздочки можно уже убрать. На этот поток была задача поменять вид критериев. Теперь есть возможность их изменять

nikolai-shabalin avatar Jan 16 '24 17:01 nikolai-shabalin

  • BUILD-01 — указывает, что сборка должна быть сделана на gulp. Но никто не будет переделывать ни на что другое. Это сложно, даже я не пытаюсь пока. Проще шаблон сборки переписывать тогда уже силами всех заинтересованных (коих не много но всё же). Критерий не несёт никакой пользы, только заставит себя переписать в дальнейшем, когда сборщик таки поменяется.
  • BUILD-02 — не защищает ни от чего, от чего бы не защищали BUILD-03 и BUILD-04 вместе взятые — один требует запуска команд дев и прод сборок, второй требует, правильного состава билда. Это невозможно без наличия всех необходимых зависимостей в package.json.
  • BUILD-05 — это норм, потому что шаловливые ручки могут и удалить из игнора билд.

Итого: нужны последние три, и то скорее всего лучше что-то уже начинать поправлять в них. А первые два — просто рудименты.

firefoxic avatar Jan 23 '24 17:01 firefoxic

Build-01 При мне переделывали на webpack Build-02 Не вижу ничего плохого в этом критерии. Пускай защищает так как умеет. Главное ничему не мешает

nikolai-shabalin avatar Jan 23 '24 19:01 nikolai-shabalin