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

kakoune: add support for plugins #1356

Closed

Conversation

jcpetruzza
Copy link
Contributor

@jcpetruzza jcpetruzza commented Jun 24, 2020

Description

The kakoune editor has a plugin mechanism and several plugins are
already packaged under pkgs.kakounePlugins. However, adding these
packages to home.packages is not enough: the kakoune package
needs to be configured with the list of plugins to include, so
that they get sourced on start-up.

We add a programs.kakoune.plugins option, analogous to
programs.vim.plugins.

The change is backwards compatible since pkgs.kakoune is defined as

wrapKakoune kakoune-unwrapped { };

and wrapKakoune defaults the list of plugins to empty.

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.

  • If this PR adds a new module

@jcpetruzza jcpetruzza requested a review from rycee as a code owner June 24, 2020 10:00
The kakoune editor has a plugin mechanism and several plugins are
already packaged under `pkgs.kakounePlugins`. However, adding these
packages to `home.packages` is not enough: the `kakoune` package
needs to be configured with the list of plugins to include, so
that they get sourced on start-up.

We add a `programs.kakoune.plugins` option, analogous to
`programs.vim.plugins`.

The change is backwards compatible since `pkgs.kakoune` is defined as

```
wrapKakoune kakoune-unwrapped { };
```

and `wrapKakoune` defaults the list of plugins to empty.
@jcpetruzza
Copy link
Contributor Author

Amended the commit to add tests for this functionality

rycee pushed a commit that referenced this pull request Aug 13, 2020
The kakoune editor has a plugin mechanism and several plugins are
already packaged under `pkgs.kakounePlugins`. However, adding these
packages to `home.packages` is not enough: the `kakoune` package needs
to be configured with the list of plugins to include, so that they get
sourced on start-up.

We add a `programs.kakoune.plugins` option, analogous to
`programs.vim.plugins`.

The change is backwards compatible since `pkgs.kakoune` is defined as

    wrapKakoune kakoune-unwrapped { };

and `wrapKakoune` defaults the list of plugins to empty.

PR #1356
@rycee
Copy link
Member

rycee commented Aug 13, 2020

Thanks for the contribution! Rebased to master in 96e2f1b.

@rycee rycee closed this Aug 13, 2020
malte-v pushed a commit to malte-v/home-manager that referenced this pull request Feb 24, 2021
The kakoune editor has a plugin mechanism and several plugins are
already packaged under `pkgs.kakounePlugins`. However, adding these
packages to `home.packages` is not enough: the `kakoune` package needs
to be configured with the list of plugins to include, so that they get
sourced on start-up.

We add a `programs.kakoune.plugins` option, analogous to
`programs.vim.plugins`.

The change is backwards compatible since `pkgs.kakoune` is defined as

    wrapKakoune kakoune-unwrapped { };

and `wrapKakoune` defaults the list of plugins to empty.

PR nix-community#1356
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

2 participants