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

Add support for Russian, Ukrainian and Bulgarian #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vlevit
Copy link

@vlevit vlevit commented Mar 1, 2016

No description provided.

@To1ne
Copy link

To1ne commented Sep 7, 2016

I am not the maintainer of this project, but if you would want this to get merged, I suggest you fix the build issues:

Lines longer than 80 characters: #717, #741, #765

@vlevit
Copy link
Author

vlevit commented Sep 8, 2016

Those 3 lines are source URLs in comments. I believe URLs must not be wrapped. I can make those lines fit 80 margin but formatting will look uglier. The proper fix would be to improve linter not to give false positives on strings that can't be wrapped (fill-paragraph don't wrap them) or at least disable fill-column in linter. So what now would you want me to do to get this merged?

@To1ne
Copy link

To1ne commented Sep 8, 2016

Sorry, I missed that!

You only can wait for @nschum to merge or comment...

@vlevit
Copy link
Author

vlevit commented Sep 12, 2016

Thanks! I have contacted @nschum, now waiting for the response.

stasvlasov pushed a commit to stasvlasov/auto-dictionary-mode that referenced this pull request Aug 17, 2018
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 this pull request may close these issues.

2 participants