Skip to content

neovim/coc: fix loading CoC plugin#2801

Merged
sumnerevans merged 1 commit intomasterfrom
fix-coc-load
Mar 18, 2022
Merged

neovim/coc: fix loading CoC plugin#2801
sumnerevans merged 1 commit intomasterfrom
fix-coc-load

Conversation

@sumnerevans
Copy link
Copy Markdown
Contributor

Description

Specifically, refactored all places that use cfg.plugins to use a new combined list that includes CoC if it's enabled.

Closes #2386

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

Specifically, refactored all places that use cfg.plugins to use a new
combined list that includes CoC if it's enabled.
@sumnerevans sumnerevans requested review from berbiche and teto March 18, 2022 02:07
@sumnerevans sumnerevans requested a review from rycee as a code owner March 18, 2022 02:07
@sumnerevans
Copy link
Copy Markdown
Contributor Author

FYI @izik1

Copy link
Copy Markdown
Member

@berbiche berbiche left a comment

Choose a reason for hiding this comment

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

LGTM!

@sumnerevans sumnerevans merged commit 590da80 into master Mar 18, 2022
jficz pushed a commit to jficz/home-manager that referenced this pull request Apr 7, 2022
Specifically, refactored all places that use cfg.plugins to use a new
combined list that includes CoC if it's enabled.
teto pushed a commit to teto/home-manager that referenced this pull request Aug 22, 2022
Specifically, refactored all places that use cfg.plugins to use a new
combined list that includes CoC if it's enabled.
@teto teto mentioned this pull request Aug 22, 2022
7 tasks
spacekookie pushed a commit to spacekookie/home-manager that referenced this pull request Feb 10, 2023
Specifically, refactored all places that use cfg.plugins to use a new
combined list that includes CoC if it's enabled.
@khaneliman khaneliman deleted the fix-coc-load branch April 5, 2025 19:53
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.

bug: programs.neovim.coc.enable dosen't load CoC

2 participants