-
-
Notifications
You must be signed in to change notification settings - Fork 164
python/uv: init #978
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
base: main
Are you sure you want to change the base?
python/uv: init #978
Conversation
|
sorry clicked the evil github update branch button give me a second to force push to rebase onto main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I don't use Python, or uv, myself but I'm sure this'll be useful to have.
I'd like to ask you to move it under vim.languages.python.extensions, you can take a look at the Markdown and Typst module to see an example but it's really straightforward. LGTM otherwise.
| inherit (cfg.uv) setupOpts; | ||
| # Plugin should be loaded whenever we enter a buffer | ||
| # so it can load the uv virtual environment. | ||
| event = ["BufEnter"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not FileType python? or even BufEnter *.py if there are event order issues
This PR adds uv.nvim, which adds the UV package manager functionality, primarily being able to activate virtual environments inside neovim for use with LSPs.
i added it as languages.python.uv.* but it could easily fit under vim.utility as well - let me know if I should make this change
Sanity Checking
nix fmt).#nix(default package).#maximal.#docs-html(manual, must build).#docs-linkcheck(optional, please build if adding links)x86_64-linuxaarch64-linuxx86_64-darwinaarch64-darwinAdd a 👍 reaction to pull requests you find important.