Skip to content

feat: add signature help - #869

Open
sophiathedev wants to merge 1 commit into
expert-lsp:mainfrom
sophiathedev:issue/142
Open

feat: add signature help#869
sophiathedev wants to merge 1 commit into
expert-lsp:mainfrom
sophiathedev:issue/142

Conversation

@sophiathedev

Copy link
Copy Markdown
Contributor

Summary

  • expose ElixirSense signature data through the project Engine API
  • handle textDocument/signatureHelp with active parameters, documentation, specs, and selected metadata
  • advertise ( and , as signature-help trigger characters

Testing

  • Engine targeted tests: 3 passed
  • Expert targeted tests: 3 passed
  • Engine full suite: 1164 passed, 19 excluded
  • Expert full suite: 844 passed, 12 excluded
  • mix compile --warnings-as-errors, mix format --check-formatted, mix credo --strict, and mix dialyzer in both applications

Closes #142

Expose ElixirSense signatures through textDocument/signatureHelp.

Closes expert-lsp#142
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.

Signature Help

1 participant