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

A word with hyphen is not being suggested by spell checker. #58

Closed
AbhilashR2020 opened this issue Aug 7, 2020 · 4 comments · Fixed by #59
Closed

A word with hyphen is not being suggested by spell checker. #58

AbhilashR2020 opened this issue Aug 7, 2020 · 4 comments · Fixed by #59

Comments

@AbhilashR2020
Copy link

Hi Team,

I have tested this sheldon spell checker utility. I did see, the the hyphen words are not suggested by the spell checker.
for example:
self-ownership
self-paid
self-painter
self-pampered
self-pampering
self-panegyric
self-parasitism
self-parricide
self-partiality
self-paying
self-peace

Is there something i am missing to validate this?.

@ferigis
Copy link
Member

ferigis commented Aug 7, 2020

hi @AbhilashR2020,

Thanks for using Sheldon! :)

I am not sure if I am following you, I just tested this misspelled word and it seems it works fine:

6> sheldon:check("self-ownersheep").
#{bazinga =>
      <<"The only way you'd be able to make a contribution to science is if they resume sending chimps into space.">>,
  misspelled_words =>
      [#{candidates => ["self-ownership"],
         line_number => 1,word => "self-ownersheep"}]}

What is exactly the issue you are facing?

@AbhilashR2020
Copy link
Author

Thanks for your quick response.
Please find a sample what i am trying out.

sheldon:check("selfownership").
#{bazinga =>
<<"The correct animal for inter-species super soldier is the koala. You would wind up with an army so cute, it coul"...>>,
misspelled_words =>
[#{candidates => [],line_number => 1,word => "selfownership"}]}
can i get self-ownership as candidate?

@ferigis
Copy link
Member

ferigis commented Aug 7, 2020

oh I understand now. Let me check

@ferigis
Copy link
Member

ferigis commented Aug 7, 2020

@AbhilashR2020 we just merged the fix, so it should work now on master. Today we will release a new version with the fix on hex.pm. Thanks!

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

Successfully merging a pull request may close this issue.

2 participants