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

lsp: implement cueimports behaviour under config for Format #3573

Open
myitcv opened this issue Nov 14, 2024 · 0 comments
Open

lsp: implement cueimports behaviour under config for Format #3573

myitcv opened this issue Nov 14, 2024 · 0 comments
Labels
FeatureRequest New feature or request lsp

Comments

@myitcv
Copy link
Member

myitcv commented Nov 14, 2024

Go has goimports:

goimports updates your Go import lines, adding missing ones and removing unreferenced ones.

Indeed this behaviour is also available via gopls via a config option.

CUE should have something similar.

Prior art here is @asdine's https://github.com/asdine/cueimports.

We should provide a config option and mode for this in cue lsp.

Note: this will require that cache of parsed artefacts, because the local resolution must inspect files in the same package to verify that it is unresolved.

@myitcv myitcv added FeatureRequest New feature or request lsp labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FeatureRequest New feature or request lsp
Projects
None yet
Development

No branches or pull requests

1 participant