Skip to content

Ложное срабатывание doc-comment-description-ends-on-dot для секции #1493

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

Merged
merged 1 commit into from
May 19, 2025

Conversation

ukolabrother
Copy link
Collaborator

Что сделано

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

Чек-лист

Общее:

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

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

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

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

Closes #??

* Добавить сценарий в тестовый кейс
* Количество маркеров и их положение не изменяется
@ukolabrother ukolabrother self-assigned this May 14, 2025
@AlmazNasibullin AlmazNasibullin changed the base branch from master to edt-2025-1 May 14, 2025 09:21
private boolean isParentExampleSection(BslDocumentationComment root, TextPart lastPart)
{
return root.getExampleSection() != null
&& root.getExampleSection().getDescription().getParts().contains(lastPart);
Copy link
Collaborator

Choose a reason for hiding this comment

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

А getDescription() или getParts() не могут null вернуть?
Я контракт не смотрел, просто уточняю, не нужно ли тут проверить на null.

Choose a reason for hiding this comment

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

По контракту не могут

@AlmazNasibullin AlmazNasibullin merged commit 39edd01 into edt-2025-1 May 19, 2025
1 of 2 checks passed
@AlmazNasibullin AlmazNasibullin deleted the G5V8DT-26943 branch May 19, 2025 21:58
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.

5 participants