Skip to content

Add support for Russian, Ukrainian and Bulgarian#12

Open
vlevit wants to merge 1 commit intonschum:masterfrom
vlevit:ru-uk-bg
Open

Add support for Russian, Ukrainian and Bulgarian#12
vlevit wants to merge 1 commit intonschum:masterfrom
vlevit:ru-uk-bg

Conversation

@vlevit
Copy link
Copy Markdown

@vlevit vlevit commented Mar 1, 2016

No description provided.

@To1ne
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown

To1ne commented Sep 8, 2016

Sorry, I missed that!

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

@vlevit
Copy link
Copy Markdown
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