bsl-language-server icon indicating copy to clipboard operation
bsl-language-server copied to clipboard

Published 20 hours ago • verified publisher icon 1c-syntax

Реалиация inlayHints для вызовов методов из конфигурации

Open nixel2007 opened this issue 3 years ago • 3 comments

Описание

Драфт. Для работы требуется insider-сборка vscode с обновленным vscode-languageclient до 8.0.0-next14

image

Доделать:

  • [ ] Оптимизировать многократное получение списка узлов дерева с вызовами. Возможно развернуть логику (от doCall к references, а как сейчас) или сделать мерж коллекций doCall и коллекций references
  • [ ] Отключение хинтов вообще
  • [ ] Отключение хинта для значений по умолчанию
  • [ ] опция на принудительный вывод имени параметра (даже если имя совпадает)
  • [ ] Перенести тесты из тестов провайдера в тесты сапплайера
  • [ ] Поддержка inlayHint/resolve
  • [x] Подумать, может быть где-то нужно сделать поддержку inlayHint/refresh. Хотя бы по аналогии с линзами, при изменении конфига.
  • [x] Тултип с описанием параметра при наведении на хинт
  • [ ] javadoc
  • [ ] Документация по конфигурационному файлу
  • [ ] Обновление схемы конфигурационного файла

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

Closes

Чеклист

Общие

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

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

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

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

nixel2007 avatar Mar 28 '22 13:03 nixel2007

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 32 Code Smells

74.4% 74.4% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Apr 10 '22 13:04 sonarqubecloud[bot]

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 27 Code Smells

74.4% 74.4% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Jul 19 '22 21:07 sonarqubecloud[bot]

На данный момент хинты не выводятся в случае если в передаваемом значении (имени переменной\имени метода) содержится само имя параметра, хотелось бы данное поведение сделать опциональным, т.е что бы была опция которая позволила бы выводить хинты всегда

sfaqer avatar Jul 19 '22 22:07 sfaqer

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 27 Code Smells

74.4% 74.4% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Oct 26 '22 07:10 sonarqubecloud[bot]

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 27 Code Smells

74.4% 74.4% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Dec 05 '22 18:12 sonarqubecloud[bot]

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

ghost avatar Apr 01 '23 19:04 ghost

~~https://sonarcloud.io/component_measures?metric=new_coverage&selected=1c-syntax_bsl-language-server%3Asrc%2Fmain%2Fjava%2Fcom%2Fgithub%2F_1c_syntax%2Fbsl%2Flanguageserver%2Finlayhints%2Finfrastructure%2FInlayHintsConfiguration.java&view=list&id=1c-syntax_bsl-language-server~~

~~Судя по кавереджу, инжект включённых сапплаеров не работает. Нужно написать тест на провайдер и/или конфигурацию и исправить квалификаторы, если это действительно так.~~

Готово.

nixel2007 avatar May 14 '23 14:05 nixel2007

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 6 Code Smells

88.1% 88.1% Coverage
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar May 30 '23 06:05 sonarqubecloud[bot]