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

Условный критерий (для слайдов) -если есть ссылка на гитхаб или его аналоги, проверить что она не 404 #444

Closed
zmm opened this issue Jun 2, 2023 · 2 comments · Fixed by #452
Assignees

Comments

@zmm
Copy link
Contributor

zmm commented Jun 2, 2023

если есть ссылка или упоминания репо

@MarinaProsche
Copy link
Collaborator

https://github.com/moevm/mse_auto_checking_slides_vaganov/tree/444_verify_git_link
Первый вариант проверки слайдов на корректность репозиториев.

Реализованы:

в слайде "Апробация"
Поиск репозиториев github, gitlab, bitbucket.
Поиск некорректных выражений вместо ссылок, например "(github)" или "gitlab"
Отсутствие ссылок при упоминании репозитория .

Во всем проекте:
Репозитории "github" проверяются на существование, пустоту (парсингом), и на приватность.
Репозитории "gitlab" проверяются на существование и приватность.
Репозитории "bitbucket" проверяются на существование, из проверки на приватность исключены, поскольку недоступны в России.

Сейчас для проверки репозитория "gitlab" использован личный токен, нужна консультация, как подкорректировать.
Для более четкой проверки всех репозиториев на пустоту было решено использовать отдельную проверку.

@HadronCollider
Copy link
Collaborator

HadronCollider commented Jun 20, 2023

Добавить в PresVerifyGitLinkCheck параметр "deep_check (bool)"

  • если False (по умолчанию) - осуществлять "простую" проверку - что код возврата ссылок = 200 (иначе - в список некорректных)
  • если True - логика с глубокой проверкой репозиториев (кол-во файлов) - данный код вынести в отдельный метод для возможного дальнейшего расширения (проверка кол-ва коммитов / их историю и пр.)

От gitlab пока можно отказаться (закомментируйте, чтобы код остался на будущее)

@HadronCollider HadronCollider linked a pull request Jun 20, 2023 that will close this issue
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 a pull request may close this issue.

3 participants