Skip to content

feat: macro autocomplete has descriptions#4712

Merged
benfdking merged 1 commit intomainfrom
codex/add-extra-details-to-autocomplete-suggestions
Jun 11, 2025
Merged

feat: macro autocomplete has descriptions#4712
benfdking merged 1 commit intomainfrom
codex/add-extra-details-to-autocomplete-suggestions

Conversation

@benfdking
Copy link
Copy Markdown
Contributor

@benfdking benfdking commented Jun 11, 2025

image

Summary

  • expose MacroCompletion with optional docstring
  • include macro descriptions when building LSP completions
  • surface macro docs as completion item documentation
  • adjust tests for new structure

Testing

  • pre-commit run --files sqlmesh/lsp/completions.py sqlmesh/lsp/custom.py sqlmesh/lsp/main.py tests/lsp/test_completions.py
  • pytest tests/lsp/test_completions.py::test_get_macros -q

https://chatgpt.com/codex/tasks/task_e_68493f049d348330899ab339336e981a

Comment thread sqlmesh/lsp/custom.py Outdated
Comment thread tests/lsp/test_completions.py Outdated
@benfdking benfdking force-pushed the codex/add-extra-details-to-autocomplete-suggestions branch from 8d4f871 to fcfa8ef Compare June 11, 2025 09:12
@benfdking benfdking changed the title Enhance LSP macro autocomplete feat: macro autocomplete has descriptions Jun 11, 2025
@benfdking benfdking enabled auto-merge (squash) June 11, 2025 09:15
@benfdking benfdking merged commit 7e252f8 into main Jun 11, 2025
24 checks passed
@benfdking benfdking deleted the codex/add-extra-details-to-autocomplete-suggestions branch June 11, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants