Skip to content

Add plugin hook for documentSymbol#1116

Merged
TwitchBronBron merged 3 commits into
masterfrom
plugin-hook-documentSymbol
Mar 19, 2024
Merged

Add plugin hook for documentSymbol#1116
TwitchBronBron merged 3 commits into
masterfrom
plugin-hook-documentSymbol

Conversation

@TwitchBronBron

@TwitchBronBron TwitchBronBron commented Mar 19, 2024

Copy link
Copy Markdown
Member
  • Adds new plugin hooks for providing document symbols.
  • Migrates the logic out of BrsFile and into BscPlugin
  • Adds a few missing document symbol entries
  • Remove the leading namespace values from the symbols since they're shown in grey next to the symbols

Before:
image

After:
image

Fixes rokucommunity/vscode-brightscript-language#385

@TwitchBronBron
TwitchBronBron merged commit a3c2ecd into master Mar 19, 2024
@TwitchBronBron
TwitchBronBron deleted the plugin-hook-documentSymbol branch March 19, 2024 17:28
TwitchBronBron added a commit that referenced this pull request May 13, 2024
* Add plugin hook for documentSymbol (#1116)

* Add plugin hook for documentSymbol

* Better handling of null/missing name tokens

* fix lint issues

* Upgade LSP packages (#1117)

* Plugin hook provide workspace symbol (#1118)

* Move symbol creation to symbolUtils

* Fix workspaceSymbol handling

* Fix tests

* Fix lint issues

* Improve workspace/document symbol handling (#1120)

* Move symbol creation to symbolUtils

* Fix workspaceSymbol handling

* Fix tests

* Fix lint issues

* Mostly fixed tests

* Fix workspace symbols missing containerName

* Update changelog for v0.65.27

* 0.65.27

* Upgrade to @rokucommunity/logger (#1137)

* Upgrade to @rokucommunity/logger

* Add original logger class back

* @rokucommunity/logger v0.3.4

* Fix node14 issues (#1153)

* Bump tar from 6.1.13 to 6.2.1 in /benchmarks (#1131)

Bumps [tar](https://github.com/isaacs/node-tar) from 6.1.13 to 6.2.1.
- [Release notes](https://github.com/isaacs/node-tar/releases)
- [Changelog](https://github.com/isaacs/node-tar/blob/main/CHANGELOG.md)
- [Commits](isaacs/node-tar@v6.1.13...v6.2.1)

---
updated-dependencies:
- dependency-name: tar
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bronley Plumb <[email protected]>

* fix: resolve the stagingDir option relative to the bsconfig.json file (#1148)

* fix: resolve the stagingDir option relative to the bsconfig.json file

* chore: add unit test

* Fix windows paths

---------

Co-authored-by: Bart van den Ende <[email protected]>
Co-authored-by: Bronley Plumb <[email protected]>

* Move function calls to separate diagnostic (#1169)

* Fix formatting with logger output (#1171)

* Update changelog for v0.67.0

* 0.67.0

* Fix crash when diagnostic is missing range (#1174)

* Fix crash when diagnostic is missing range

* improve unit test

* Fix issues with cannotFindFunction

* Removed TypeChainEntry.kind (#1178)

* fixed some tests as well

* remove commented out code

* Fixed last tests

* fix lint errors

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bart van den Ende <[email protected]>
Co-authored-by: Bart van den Ende <[email protected]>
Co-authored-by: Mark Pearce <[email protected]>
Co-authored-by: Mark Pearce <[email protected]>
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.

Add enums to the various symbol providers

1 participant