-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
markdown lint #63
base: master
Are you sure you want to change the base?
markdown lint #63
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Подтяни изменения из мастера плз.
Готово. |
@lord-vesel ребейснул ветку на мастер. |
спасибо, постараюсь найти способ автоматизированно прочекать, что ничего не ломает |
Думаю что это можно качественно прочекать только скриншотными тестами, ну или какими-нибудь снапшотами html, и оба варианта выглядят достаточно громоздко. Предлагаю move fast and break things, тем более что это просто линтер который один раз пробегает по файлам. |
typical developer Тут не только просто линтер, есть правки текстовые, приходится всё проверять. |
@alex-mos в общем, линтер добавляет кучу лишних строк и я подзадолбался их выковыривать, сделать бы с этим что-то массово |
Какие строки ты имеешь в виду? Я пробежался по диффу — он скорее поудалял много лишних строк. Если где-то добавляет строки, которые ломают вёрстку, то я его подкручу чтобы этого не делал. |
А выше в ревью, запрошенные изменения с комментами. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
нужно пофиксить списки, которые понаделал линтер из абзацев, стартующих с дефиса
Исправил места где линтер создавал лишние списки, отсмотрел глазами весь дифф в поисках подозрительных мест, проверил на сайте, внёс дополнительные правки по ходу ревью. |
No description provided.