Skip to content

Commit

Permalink
Replace company-capf with company-nimsuggest in nim-mode.
Browse files Browse the repository at this point in the history
As described here: nim-lang/nim-mode#159,
company-capf makes nim-mode unusable. I am replacing it with
company-nimsuggest as it is much more responsive. The reason I elected not to
leave company-capf in as a backup is that it would somehow still make it to the
car of company-backends, regardless of position at assignment, thus becoming the
default completion backend. Given that no functionality is lost this way, and
that nimsuggest is now part of the core nim installation
(https://github.com/nim-lang/nimsuggest), I do not think anything has been
sacrificed.
  • Loading branch information
cyruseuros authored and stephanschubert committed Oct 23, 2019
1 parent e5f1ab8 commit cf7a4bf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGELOG.develop
Original file line number Diff line number Diff line change
Expand Up @@ -1556,6 +1556,8 @@ Other:
**** Nim
- Add key binding ~SPC m h h~ to show symbol documentation (thanks to Valts
Liepiņš)
- Replace =company-capf= with =company-nimsuggest= in
=company-backends-nim-mode= to improve responsiveness (thanks to Uroš Perišić)
**** Node
- Use add-node-modules-path to automatically find local executables (thanks to
jupl)
Expand Down
2 changes: 1 addition & 1 deletion layers/+lang/nim/packages.el
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

(defun nim/post-init-company ()
(spacemacs|add-company-backends
:backends company-capf
:backends company-nimsuggest
:modes nim-mode nimscript-mode))

(defun nim/post-init-flycheck ()
Expand Down

0 comments on commit cf7a4bf

Please sign in to comment.