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

use flyspell-correct instead of helm-flyspell #6000

Closed

Conversation

@d12frosted
Copy link
Collaborator

@d12frosted d12frosted commented May 6, 2016

flyspell-correct plays nicely with custom front-ends, and it has support of ivy and helm out of box

flyspell-correct plays nicely with custom front-ends, and it has support
of ivy and helm out of box
@d12frosted
Copy link
Collaborator Author

@d12frosted d12frosted commented May 6, 2016

Wow. This is #6000 ! This is first time I scored great number :)

@d12frosted
Copy link
Collaborator Author

@d12frosted d12frosted commented May 11, 2016

😢 🐊 😿

@syl20bnr syl20bnr added Merging... Merged and removed Merging... labels May 12, 2016
@syl20bnr
Copy link
Owner

@syl20bnr syl20bnr commented May 12, 2016

Great! 🐊 is in the box 👍
Thank you !
Cherry-picked into develop branch, you can safely delete your branch.

@syl20bnr syl20bnr closed this May 12, 2016
@syl20bnr syl20bnr removed the Merged label May 12, 2016
@syl20bnr
Copy link
Owner

@syl20bnr syl20bnr commented May 12, 2016

Also there will be an achievement for the 6000th issue :-D

@d12frosted d12frosted deleted the d12frosted:features/flyspell-correct branch May 12, 2016
@d12frosted
Copy link
Collaborator Author

@d12frosted d12frosted commented May 12, 2016

Wow, you was in berserk mode, cherry-picked so much 😸 Thanks for letting 🐊 in 😸 😈

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

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.