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
base: master
Are you sure you want to change the base?
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.
Вопросы по самой идее.
О каком хабе идет речь? Есть пример подобного подхода? |
Посмотри реализацию doc-comment-hub - в EDT.
Это отдельная проверка которая включается/выключается. Для инлайн и
произвольных комментариев - нужно делать такие же хабы.
Это также возможность для пользователей отключить валидацию в тех типах
подчиненных моделей которые им не нужны.
Имхо, нужно делать 2 дополнительных хаба (или один дополнительный но с 2
параметрами) которые загружают модель (парсят) и отправляют на валидацию:
doc-comment-inline-hub - только для типизирующих комментариев которые в
строке с переменными - это явно влияющие комментарии на типизацию (коммент
добавляется в виртуальную секцию Return)
doc-comment-arbitrary-hub - для всех других комментариев в модуле, которые
парсятся и загружаются в модель док-комментов "как есть" - то есть
фактически будут в docComment.getDescription()
По правильному, эти хабы нужно создавать в ЕДТ, но можно и в Код-стайле.
Тогда все остальные проверки по модели док-комментов - будут автоматически
запускаться на этих моделях.
|
Что сделано
Реализована валидация несуществующих ссылок в комментариях в теле метода (инлайн документирующие и свободные)
Чек-лист
Общее:
master
и нет конфликтовЕсли применимо:
Закрываемые задачи
LT 1011
Closes #