Skip to content

Conversation

@artbear
Copy link
Contributor

@artbear artbear commented Sep 18, 2021

Описание

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

Closes #1822

Чеклист

Общие

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

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

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

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

@artbear
Copy link
Contributor Author

artbear commented Sep 24, 2021

@nixel2007 и в этом ПР все замечания устранены. Посмотри, пожалуйста!

@theshadowco
Copy link
Member

@artbear можешь обновиться из девелопа? там новый парсер

@asosnoviy
Copy link
Member

А что значит "неверное"?

@asosnoviy
Copy link
Member

Есть предложение избегать размытых "неверное, неточное". Что бы из названия было понятно про что диагностика. Тут похоже речь про обращение к несуществующим метаданным

@artbear
Copy link
Contributor Author

artbear commented Oct 7, 2021

@artbear можешь обновиться из девелопа? там новый парсер

да, сделаю

@artbear
Copy link
Contributor Author

artbear commented Oct 7, 2021

Есть предложение избегать размытых "неверное, неточное". Что бы из названия было понятно про что диагностика. Тут похоже речь про обращение к несуществующим метаданным

Наименование правила "Обращение к несуществующим метаданным" устроит?

@nixel2007
Copy link
Member

@artbear ... в запросе

# Conflicts: #	docs/diagnostics/index.md #	docs/en/diagnostics/index.md #	src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/parameters-schema.json #	src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/schema.json
@artbear
Copy link
Contributor Author

artbear commented Oct 8, 2021

@theshadowco @asosnoviy @nixel2007

  • Имя правила изменил на Обращение к несуществующим метаданным в запросе
  • обновился из девелопа, исправил код правила из-за нового парсера запросов
  • перезапустил прекоммит и исправил конфликты

проверяйте

@nixel2007 nixel2007 merged commit 9f5c056 into 1c-syntax:develop Oct 8, 2021
@artbear artbear deleted the WrongMetadataInQuery branch October 8, 2021 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants