Skip to content

refactor: prefer local alias for vim.api, vim.lsp#3564

Merged
clason merged 1 commit intomasterfrom
fix/alias
Nov 7, 2025
Merged

refactor: prefer local alias for vim.api, vim.lsp#3564
clason merged 1 commit intomasterfrom
fix/alias

Conversation

@clason
Copy link
Contributor

@clason clason commented Nov 6, 2025

  • be consistent
  • use longer name that isn't as likely to shadow
  • rule of thumb: > 5 occurrences merits an alias

@just1602
Copy link

just1602 commented Nov 6, 2025

Ain't no maintainer, but I reviewed it and LGTM!

I'm really happy you're doing some cleanup @clason I was really not in the mood of switching to another picker, especially since fzf-lua doesn't support the send to qflist feature that I use a lot.

@clason
Copy link
Contributor Author

clason commented Nov 6, 2025

Thanks, but don't get used to it; this is just a slap of paint and a bandaid to keep the ball rolling at least for the Nvim 0.12 release. The project itself is still on life support (but no longer critical), with its future under discussion.

@clason clason merged commit 01419c1 into master Nov 7, 2025
12 checks passed
@clason clason deleted the fix/alias branch November 7, 2025 07:48
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.

2 participants