Skip to content
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

Добавляет проверку ссылок с помощью инструмента Link Checker #228

Closed
wants to merge 10 commits into from

Conversation

igsekor
Copy link
Contributor

@igsekor igsekor commented Apr 7, 2021

Ранее в #194 было предложено использовать вот это решение. Оказалось, что нельзя пробросить настройки для инструмента внутрь готового GitHub Action. Применение инструмента напрямую выявило его неспособность работы с якорями. Все это побудило к поиску другого инструмента.

Прошу любить и жаловать. Все настройки проверки ссылок хранятся отдельно в файле .linkcheckerrc. Сделал отдельный скрипт и добавил в общий скрипт для тестирования. Также соорудил свой Action, который запускает этот скрипт при проверки любого пуша. Разумеется эту настройку надо будет поменять на нужную в момент слияния.

Приходите, пожалуйста, @pepelsbey

@pepelsbey
Copy link
Member

Немного подкрутил :) У меня пока такое вываливается при тестах:

> links
> link-checker 'dist'

WTF, this should not happen
TypeError: Cannot read property 'url' of undefined
    at getOverrideFor (/Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:59:59)
    at /Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:359:45
    at Array.forEach (<anonymous>)
    at /Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:358:14
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Walker.<anonymous> (/Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:344:3)
WTF, this should not happen
TypeError: Cannot read property 'request' of undefined
    at /Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:413:60
    at Array.forEach (<anonymous>)
    at /Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:412:14
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Walker.<anonymous> (/Users/pepelsbey/Documents/Веб-стандарты/Код/web-standards.ru/node_modules/link-checker/link-checker.js:402:3)

1154 files were scanned in dist
0 local links and 0 parent links and 5544 remote links were checked
0 errors and 0 warnings

@igsekor
Copy link
Contributor Author

igsekor commented Apr 15, 2021

@pepelsbey вот какой ишью и пиар нашел по этому поводу :(

Ищем еще вариант? Пускаем как есть с ворнингом? Ждем, пока они не решат проблему? Помогаем им ее решить? :)

@pepelsbey
Copy link
Member

Я бы подождал немного, это не то, чтобы критикал

@pepelsbey pepelsbey closed this Jun 18, 2021
@pepelsbey pepelsbey deleted the branch web-standards-ru:master June 18, 2021 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants