Skip to content

Conversation

@alonthedark
Copy link
Contributor

Что сделано

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

Чек-лист

Общее:

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

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

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

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

Closes #1508

@alonthedark alonthedark changed the title Feature/issue 1508 semicolon extra and mising Feature/issue 1508 semicolon extra and missing Oct 20, 2025
Проверка на нахождение statement
contain -> contains
ReplaceEdit -> InsertEdit
alonthedark and others added 4 commits October 29, 2025 13:58
Copy link
Collaborator

@MaksimDzyuba MaksimDzyuba left a comment

Choose a reason for hiding this comment

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

Нужно больше тестов, нужно поправить ошибку, что я нашел, не верно применена идея с "statement.eContainingFeature().isMany() && statement.eContainer() != null
&& statement.eContainer().eGet(statement.eContainingFeature()) instanceof List<?> statementCollection" ее нужно в метод checkSemicolon, и пропускать через нее хотя бы случаи

  1. else if (eObject instanceof IfStatement || eObject instanceof ForStatement)
  2. else if (eObject instanceof Statement & !(eObject instanceof EmptyStatement))
  3. else if (eObject instanceof SimpleStatement)

@kuzyara
Copy link
Contributor

kuzyara commented Dec 3, 2025

@MaksimDzyuba Я тоже хочу проверки писать на джава

@alonthedark помоги среду настроить

@MaksimDzyuba
Copy link
Collaborator

@kuzyara
Здравствуйте, плагин открытый, можете спокойно писать, будем стараться не затягивать с ревью

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.

3 participants