Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

buf beta lsp spins on empty files #3362

Closed
emcfarlane opened this issue Oct 1, 2024 · 1 comment · Fixed by #3370
Closed

buf beta lsp spins on empty files #3362

emcfarlane opened this issue Oct 1, 2024 · 1 comment · Fixed by #3370
Assignees
Labels
Bug Something isn't working

Comments

@emcfarlane
Copy link
Contributor

emcfarlane commented Oct 1, 2024

GitHub repository with your minimal reproducible example (do not fill out this field with "github.com/bufbuild/buf" or we will automatically close your issue, see the instructions above!)

https://github.com/bufbuild/registry-proto

Commands

echo "syntax = \"proto3\";" > buf/registry/module/v1/commit.proto
nvim buf/registry/module/v1/commit.proto

Output

Blocks the LSP server. Starting from the empty file causes the the SemanticTokensFull to spin. Adding a message doesn't fix it unless we also restart the LSP server with the now non-empty proto file.

Expected Output

Return an empty response if the file is empty.

Anything else?

No response

@emcfarlane emcfarlane added the Bug Something isn't working label Oct 1, 2024
@bufdev
Copy link
Member

bufdev commented Oct 2, 2024

Spinning there should really be removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants