dcs-doc icon indicating copy to clipboard operation
dcs-doc copied to clipboard

markdown lint

Open alex-mos opened this issue 3 years ago • 9 comments

alex-mos avatar Jan 31 '22 08:01 alex-mos

Готово.

alex-mos avatar Feb 03 '22 17:02 alex-mos

@lord-vesel ребейснул ветку на мастер.

alex-mos avatar Sep 07 '22 14:09 alex-mos

@lord-vesel ребейснул ветку на мастер.

спасибо, постараюсь найти способ автоматизированно прочекать, что ничего не ломает

lord-vesel avatar Sep 07 '22 20:09 lord-vesel

Думаю что это можно качественно прочекать только скриншотными тестами, ну или какими-нибудь снапшотами html, и оба варианта выглядят достаточно громоздко. Предлагаю move fast and break things, тем более что это просто линтер который один раз пробегает по файлам.

alex-mos avatar Sep 10 '22 11:09 alex-mos

move fast and break things

typical developer

Тут не только просто линтер, есть правки текстовые, приходится всё проверять.

lord-vesel avatar Sep 12 '22 19:09 lord-vesel

@alex-mos в общем, линтер добавляет кучу лишних строк и я подзадолбался их выковыривать, сделать бы с этим что-то массово

lord-vesel avatar Sep 12 '22 21:09 lord-vesel

Какие строки ты имеешь в виду? Я пробежался по диффу — он скорее поудалял много лишних строк. Если где-то добавляет строки, которые ломают вёрстку, то я его подкручу чтобы этого не делал.

alex-mos avatar Sep 13 '22 09:09 alex-mos

Какие строки ты имеешь в виду?

А выше в ревью, запрошенные изменения с комментами. Линтер добавляет строку перед строкой, начинающейся с "-", считая, что это начало списка.

lord-vesel avatar Sep 14 '22 07:09 lord-vesel

Исправил места где линтер создавал лишние списки, отсмотрел глазами весь дифф в поисках подозрительных мест, проверил на сайте, внёс дополнительные правки по ходу ревью.

alex-mos avatar Oct 03 '22 15:10 alex-mos