search

AtomLinter / linter-spell

Multilingual grammar-specific spell checking for Atom and linter using Ispell compatible interface such as GNU Aspell or Hunspell.
https://atom.io/packages/linter-spell
MIT License
14 stars 1 forks source link

1.12.2 Incompatibility #36

Closed ChrisChinchilla closed 7 years ago

ChrisChinchilla commented 7 years ago

I'm not 100% sure if this is a valid issue or not, but I recently updated Atom to the version released in the past couple of days, and also did some laptop spring cleaning, so maybe I deleted something.

But now I can't get spell checking to work and receive this error:

screenshot 2016-11-12 10 52 32
yitzchak commented 7 years ago

Looks like it is having a hard time running aspell? Can you check to make sure is in the search path?

ChrisChinchilla commented 7 years ago

@yitzchak I don't think so as I don't remember having this issue with aspell in Atom before, could you give an example of settings that you would expect? If so, I may make a PR against the read me to make that a bit clearer.

yitzchak commented 7 years ago

I suspect the error is coming from the call to aspell dicts which should return a list of dictionaries. I added an error message and a null check. Try to update to v0.12.4. It probably won't fix the issue, but it may tell us more.

ChrisChinchilla commented 7 years ago

@yitzchak I got his working again by specifying the complete path to aspell (i.e /usr/local/bin/aspell) and deleting all the settings for languages and dictionaries. I'm now unsure if this was a bug or not, maybe see if anyone else gets it :/

yitzchak commented 7 years ago

@ChrisChinchilla Glad you got it working again. I'll dig around and see if there is a root cause.