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

Добавить валидацию некорректных ссылок в комментариях для метода #1389

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ukolabrother
Copy link
Collaborator

Что сделано

Реализована валидация несуществующих ссылок в комментариях в теле метода (инлайн документирующие и свободные)

  • Опишите изменения функциональности
  • Реализованные проверки кода и метаданных

Чек-лист

Общее:

  • ветка PR обновлена из master и нет конфликтов
  • Тесты-кейсы, JUnit тесты правильного и неправильного состояния
  • Измененные Вами исходники отформатированы в соответствии с конвенцией
  • Авто-аудит (SonarQube и CheckStyle) пройден, покрытие кода хорошее, ошибок нет, плохой код устранен
  • Добавлена запись в ИСТОРИЮ ИЗМЕНЕНИЯ, включаемая в пользовательскую документацию плагина

Если применимо:

  • Пользовательская документация на доп.инструменты написана (на русском)
  • Описание проверок - на двух языках

Закрываемые задачи

LT 1011
Closes #

@marmyshev
Copy link
Collaborator

А в чем смысл этих изменений?

Почему убрана подписка на объект ссылок в модели док комментов?

ПО правильному- нужно делать хаб который будет проверять инлайн-комментарии как модель, а там есть ли ссылки или типы, или ещё что-то- уже задача отдельных проверок.

Copy link
Collaborator

@marmyshev marmyshev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вопросы по самой идее.

@ukolabrother
Copy link
Collaborator Author

О каком хабе идет речь? Есть пример подобного подхода?
Если ссылка размещена в свободном комментарии или в инлайн-документирующем, то она не будет отправлена на проверку в предыдущей версии валидатора. Только документирующий комментарий над методом

@marmyshev
Copy link
Collaborator

marmyshev commented Dec 12, 2023 via email

@marmyshev marmyshev marked this pull request as draft December 16, 2023 00:25
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.

None yet

2 participants