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

[MOD] UnusedLocalMethod - "Неиспользуемый локальный метод" - поддержка модулей объектов через параметр правила #3111

Closed
artbear opened this issue Jul 23, 2023 · 2 comments · Fixed by #3112
Labels
component/diagnostics Доработка / создание диагностики type/discuss

Comments

@artbear
Copy link
Contributor

artbear commented Jul 23, 2023

Диагностика

UnusedLocalMethod - Неиспользуемый локальный метод

Описание предлагаемых изменений

Предлагаю добавить поддержку модулей объектов через булевый параметр правила
считаю, что для этих модулей правило будет полезно и ложных срабатываний не будет много, поэтому с ними можно мириться
а также полезно для внешних файлов

по умолчанию параметр выключен и проверки модулей объектов не производится
при включении параметра правило начинает работать в модулях объектов

Дополнительная информация

@artbear artbear added component/diagnostics Доработка / создание диагностики type/discuss labels Jul 23, 2023
artbear added a commit to artbear/bsl-language-server that referenced this issue Jul 23, 2023
по умолчанию, как и сейчас, модули объектов игнорятся

1c-syntax#3111
artbear added a commit to artbear/bsl-language-server that referenced this issue Jul 23, 2023
@nixel2007
Copy link
Member

Не совсем понимаю тезис про "не будет фонить". Оно фонит именно на модулях объектов из-за стандартных обработчиков объектов, которые в явном виде из кода не вызываются

@artbear
Copy link
Contributor Author

artbear commented Aug 24, 2023

Не совсем понимаю тезис про "не будет фонить". Оно фонит именно на модулях объектов из-за стандартных обработчиков объектов, которые в явном виде из кода не вызываются

типовые, часто встречаемые имена, ПередЗаписью, ПриЗаписи и т.п. можно добавить в исключения
а остальные методы считать ошибкой.

в этом случае ФП будет немного.

и напомню, что такое поведение по умолчанию выключено.
кому нужно, тот настроит.

нам, в Инфостарте, нужно!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/diagnostics Доработка / создание диагностики type/discuss
Projects
None yet
2 participants