Move project to json-rpc about anaconda-mode HOT 12 CLOSED

What remains to be done here?

from anaconda-mode.

Move client to external json-rpc library. I'll look at it in the evening.

from anaconda-mode.

Unfortunately I forget about #25. We must resolve this too. And write tests for it. Python 2.6 doesn't has argparse module. So that's really required.

from anaconda-mode.

I can change argparse to click, it's a quick thing to do.

from anaconda-mode.

That would be great 👍

from anaconda-mode.

Alright, I pushed some commits. Remember to update submodules. I guess tests for starting the server can be part of #26?

from anaconda-mode.

I think to test #26 we can remove sleep-for line from test/test-helper.el file.

from anaconda-mode.

Also this require new melpa package

(anaconda-mode
 :fetcher github
 :repo "proofit404/anaconda-mode"
 :files ("*.el" "*.py" "vendor"))

from anaconda-mode.

@fbergroth some day I'll kill my sysadmin. So lets move all conversations about anaconda-mode to corresponding branch issue

I had no idea destructuring-bind was deprecated. Where can I read about that?

destructuring-bind is a part of cl.el library. cl library deprecated because it override some normal elisp functions (case for example). That's why you need (eval-when-compile (require 'cl)) instead of simple (require 'cl). In cl-lib all this function prefixed with cl- so it can be safely imported.

I'm using flycheck, but it only warns me about missing docstrings. Am I missing some configuration?

First of all update your flycheck package. If nothing happens, look at flycheck-checkers variable. It must contain emacs-lisp and emacs-lisp-checkdoc. Also flycheck-cask package is required.

One more thing. Right now all the functions are called anaconda-mode-functionname. I think we should rename it to anaconda-functionname. Also anaconda-mode.el can be renamed anaconda.el. What do you think?

After plugin system'll implemented anaconda-mode will contain json-rpc interaction only, all jedi api calls will be moved into anaconda-nav and it will renamed to anaconda-jedi.el. All function in it will be called anaconda-jedi-function-name so I don't think anaconda.el is necessary.

from anaconda-mode.

@fbergroth I think we can merge it to upstream and update melpa recipe.

from anaconda-mode.

Sure! Readme isn't updated yet though. But we can improve that piece by piece. Maybe just add new keybindings for now?

from anaconda-mode.

After plugin system I plan trim README to few lines and one screenshot, keep badges and add link to complete guide on readthedocs service. So we can simply fix some lines for now.

from anaconda-mode.