-
Notifications
You must be signed in to change notification settings - Fork 14
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
anki-editor-api-check: Failed to connect to Anki: End of file while parsing JSON #5
Comments
Yes, I see the same behavior. This is because a function |
Yes, as a minimum I think.
This makes sense. |
Increases user-friendliness, as discussed in #5.
After some consideration, I think it's best to settle with just the clearer message. Almost all commands anki-editor provides do in fact need Anki running. Adding the check to each command would add an extra API call to each, which is undesirable. I'm also consciously only adding the clearer error message to If you really want to avoid the call to |
This may not be a bug but it did not occur previously when on the
master
branch fromlouietan/anki-editor
and the following config:(The use of
:after
and:demand
here is because I lazy load my packages as much as possible.)Switching to this fork my
use-package
form becomes:So it seems with the merge of the
louietan/anki-editor:develop
branch Anki needs to be running to use the minor mode. This makes sense but it is odd I saw no such message when on theloueitan/anki-editor:master
branch. Indeed thedevelop
branch introduced changes to the API but nothing as yet stands out which would point to why I didn't see this message before.The text was updated successfully, but these errors were encountered: