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

feat: add cmp-tmux to the list of sources #2542

Merged
merged 1 commit into from May 1, 2022

Conversation

pschmitt
Copy link
Contributor

Description

This adds tmux to the cmp sources.

https://github.com/andersevenrud/cmp-tmux

How Has This Been Tested?

Tested locally, cmp-tmux works as expected.

@kylo252 kylo252 changed the title [Feature] Add cmp-tmux to the list of sources feat: add cmp-tmux to the list of sources Apr 30, 2022
Copy link
Collaborator

@kylo252 kylo252 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kylo252 kylo252 merged commit d0c5393 into LunarVim:rolling May 1, 2022
tomazursic pushed a commit to tomazursic/LunarVim that referenced this pull request May 1, 2022
* upstream/rolling:
  feat: add cmp-tmux to the list of sources (LunarVim#2542)
  feat: gitsigns yadm support (LunarVim#2535)
  fix(installer): latest and specified release version for neovim have different urls (LunarVim#2484)
  fix(lsp): undo stdpath overload to avoid datarace (LunarVim#2540)
  feat(lua-dev): use the newer lua-dev branch till folke comes back (LunarVim#2538)
  fix(nvimtree): add latest changes from nvimtree (LunarVim#2537)
  feat(icons): make it possible to disable icons (LunarVim#2529)
  refactor(lsp): decouple the installer setup-hook (LunarVim#2536)
@pschmitt pschmitt deleted the cmp-tmux branch May 1, 2022 15:00
@danielo515
Copy link
Contributor

What is the intention behind this? As far as I know ,that cmp extension is not shipped by default

@kylo252
Copy link
Collaborator

kylo252 commented Jun 11, 2022

What is the intention behind this? As far as I know ,that cmp extension is not shipped by default

it makes it so that you don't have to manually add tmux to the list of recognizable sources, so just installing the plugin would make it work fine with cmp, tabnine is another example.

@danielo515
Copy link
Contributor

danielo515 commented Jun 11, 2022 via email

@thyseus
Copy link

thyseus commented Feb 24, 2023

Another solution would be to simply ship

https://github.com/andersevenrud/cmp-tmux

with the default plugins of lunarvim. This plugin is lightweight and tmux is very common imho.

@kylo252
Copy link
Collaborator

kylo252 commented Mar 2, 2023

Another solution would be to simply ship

andersevenrud/cmp-tmux

with the default plugins of lunarvim. This plugin is lightweight and tmux is very common imho.

I think that would open the flood doors to including all of cmp's sources 🤔

@thyseus
Copy link

thyseus commented Mar 2, 2023

Thats true. Maybe a separate configuration file for cmp that has all sources listed at https://github.com/hrsh7th/nvim-cmp/wiki/List-of-sources but COMMENTED by default could be shipped with lunarvim. One just needs to uncomment the cmp plugins he is interested in ?

@kylo252
Copy link
Collaborator

kylo252 commented Mar 8, 2023

One just needs to uncomment the cmp plugins he is interested in ?

@thyseus, please create a PR to update this list with some of the missing useful/common sources

-- -- Additional Plugins <https://www.lunarvim.org/docs/plugins#user-plugins>
-- lvim.plugins = {
-- {
-- "folke/trouble.nvim",
-- cmd = "TroubleToggle",
-- },
-- }

and maybe another PR to add them to the site 😄

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.

None yet

4 participants