Skip to content

Conversation

@theshadowco
Copy link
Member

@theshadowco theshadowco commented Oct 4, 2021

Описание

  1. Реализована диагностика, сигнализирующая о наличии НЕКЛИЕНТСКИХ экспортных методах формы
  2. Добавлен метод addDiagnostic для MethodSymbol

Связанные задачи

Closes: #1289

Чеклист

Общие

  • Ветка PR обновлена из develop
  • Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
  • Изменения покрыты тестами
  • Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

  • Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

Copy link
Member

@nixel2007 nixel2007 left a comment

Choose a reason for hiding this comment

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

Описание

@theshadowco theshadowco force-pushed the feature/ExportMethodInForm branch 2 times, most recently from 58ba4f6 to cda7d7d Compare October 5, 2021 16:09
@asosnoviy asosnoviy changed the title Новая диагностика - некорректный экспортный метод формы Новая диагностика - Серверный экспортный метод формы Oct 6, 2021
Copy link
Member

@nixel2007 nixel2007 left a comment

Choose a reason for hiding this comment

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

небольшой вопрос по поводу ModuleSymbol, а так аппрув

2. Внесены правки в документацию по коду
2. дополнил `ModuleSymbol` областью на первый токен 3. скорректировал метод `addDiagnostic` для работы со всеми символами
@theshadowco theshadowco force-pushed the feature/ExportMethodInForm branch from 56a0e58 to 601d36e Compare October 7, 2021 06:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2021

@nixel2007 nixel2007 merged commit f6f2895 into develop Oct 7, 2021
@nixel2007 nixel2007 deleted the feature/ExportMethodInForm branch October 7, 2021 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment