search

bespoyasov / solidbook

Book about the SOLID principles and object-oriented software design.
https://solidbook.vercel.app
MIT License
609 stars 58 forks source link

[WIP] Добавляет линтер правописания #138

Closed baitun closed 2 years ago

baitun commented 5 years ago

Как обсудили в #137, добавил yaspeller.
Выполнить проверку всех файлов можно командой yarn spell, можно так же сгенерировать отчет в формате html с помощью yarn spell --report html.

image

Сейчас при запуске показывает много ошибок, некоторые из них действительно ошибки и опечатки, а некоторые слова яндекс.спеллер просто не знает. Нужно теперь вручную отревьювить все ошибки, и те которые действительно являются ошибки исправить, а те что нет - добавить в исключения в dictionary файл .yaspeller.json.

После этого можно будет добавить проверку в CI наравне с юнит-тестами, или в пре-коммит хук.

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://ota-solid-git-fork-baitun-feature-yaspeller.ota.now.sh

pongo commented 5 years ago

Чтобы проверять задания в книге, нужно расширить список мест: yaspeller pages/**/*.mdx components/Quiz/quizzes/**/*.mdx components/Quiz/quizzes/**/*.tsx

Но тогда туда попадет код. Чтобы уменьшить число слов можно игнорировать латиницу, добавив опцию "ignoreLatin": true в .yaspeller.json.

pongo commented 5 years ago

Обновил словарь: https://pastebin.com/mgywD7Xh

baitun commented 5 years ago

@pongo спасибо! Словарь добавил и список файлов для проверки расширил. Теперь остались только актуальные ошибки.

bespoyasov commented 5 years ago

Вау, это очень круто! :–)

@dex157 что думаешь?

dex157 commented 5 years ago

Осталось ci прикрутить. Думаю в рамках эксперимента надо попробовать github actions =)

upd: добавил нас в очередь на бэту

bespoyasov commented 4 years ago

@dex157, если я правильно понял, они вышли из беты? Давай попробуем запилить CI?

dex157 commented 4 years ago

@bespoyasov давай. Пока просто прогон тестов, tsc? Можно еще деплой на now.sh перенести на github actions, только вот нужно ли это делать, не ясно )

bespoyasov commented 4 years ago

Не уверен, что деплой стоит переносить. Предлагаю двигаться ма-а-аленькими шагами и начать с тестов, как ты предложил :–)

baitun commented 2 years ago

Закрываю, так как был созан новый MR #176