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

Spelling on python syntax? #77

Closed
s0undt3ch opened this issue Sep 2, 2014 · 8 comments · Fixed by #79
Closed

Spelling on python syntax? #77

s0undt3ch opened this issue Sep 2, 2014 · 8 comments · Fixed by #79

Comments

@s0undt3ch
Copy link
Contributor

After installing riv, I get a properly highlighted docstring, however, python variables or module names are checked for spelling!?

riv

@gu-fan
Copy link
Owner

gu-fan commented Sep 3, 2014

I checked the source file, but did not found anything touching spell settings.
can you try set nospell??
if works, then
au FileType python set nospell??

@s0undt3ch
Copy link
Contributor Author

I do want spelling on python, but before your plugin only doctring and strings were checked for spelling and now that's lost.

@s0undt3ch
Copy link
Contributor Author

Any reason whey the rst syntax is packed with @NoSpell and not an instance of @Spell is found?

@s0undt3ch
Copy link
Contributor Author

My python syntax with spell enabled file is this one, by python-mode

@gu-fan
Copy link
Owner

gu-fan commented Sep 4, 2014

you are right, let me try to add some @spell

@gu-fan gu-fan closed this as completed in 1b851fc Sep 4, 2014
@gu-fan
Copy link
Owner

gu-fan commented Sep 4, 2014

But the python part, don't know how to fix it.

@s0undt3ch
Copy link
Contributor Author

I will try your fix later on but it seems that once I enable your plugin on doctrings, the python spelling rules are lost...

I'll let you know.

@s0undt3ch
Copy link
Contributor Author

I think I found out the culprit code here. Those lines enable spelling on the whole file, which, on docstrings residing on python files is not wanted...

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