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

vimPlugins.deoplete-dictionary: init at 2019-04-16 #68464

Merged
merged 2 commits into from Sep 11, 2019

Conversation

justinlovinger
Copy link
Contributor

Motivation for this change

Add deoplete-dictionary vim plugin

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Tested vim plugin
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

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

This actually does an update as well, but thats alright.

nix-review passes on NixOS
diff LGTM

[93 built, 34 copied (824.4 MiB), 191.3 MiB DL]
https://github.com/NixOS/nixpkgs/pull/68464
47 package were build:
vimPlugins.ale vimPlugins.coc-git vimPlugins.coc-go vimPlugins.coc-pairs vimPlugins.coc-snippets vimPlugins.coc-tsserver vimPlugins.denite-git vimPlugins.denite-nvim vimPlugins.deol-nvim vimPlugins.deoplete-dictionary vimPlugins.deoplete-fish vimPlugins.deoplete-lsp vimPlugins.deoplete-nvim vimPlugins.falcon vimPlugins.fastfold vimPlugins.ghcid vimPlugins.iosvkem vimPlugins.julia-vim vimPlugins.lh-vim-lib vimPlugins.lightline-vim vimPlugins.neomake vimPlugins.neoterm vimPlugins.nerdtree vimPlugins.nvimdev-nvim vimPlugins.vim-airline vimPlugins.vim-autoformat vimPlugins.vim-elixir vimPlugins.vim-fireplace vimPlugins.vim-fugitive vimPlugins.vim-gitgutter vimPlugins.vim-go vimPlugins.vim-grepper vimPlugins.vim-jsx-pretty vimPlugins.vim-lsc vimPlugins.vim-plug vimPlugins.vim-polyglot vimPlugins.vim-rhubarb vimPlugins.vim-ruby vimPlugins.vim-sensible vimPlugins.vim-snippets vimPlugins.vim-table-mode vimPlugins.vim-test vimPlugins.vim-visual-multi vimPlugins.vimtex vimPlugins.yats-vim vimPlugins.youcompleteme vimPlugins.zig-vim

@teto
Copy link
Member

teto commented Sep 11, 2019

please split the addition of the plugin and the update into two commits (via git add -p for isntance)

@justinlovinger
Copy link
Contributor Author

This actually does an update as well, but thats alright.

I was following the official instructions, which say to create a commit with an update before adding a new plugin.

please split the addition of the plugin and the update into two commits (via git add -p for isntance)

The update and the addition are separate commits, with commit messages following the official instructions.

@teto teto merged commit b425012 into NixOS:master Sep 11, 2019
@teto
Copy link
Member

teto commented Sep 11, 2019

you are right sorry :(

@justinlovinger
Copy link
Contributor Author

@teto No problem :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants