-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
replace custom ivy hydra-menu by original hydra-menu #13377
replace custom ivy hydra-menu by original hydra-menu #13377
Conversation
👍 For this change, my only gripe is the missing |
Thank you for contributing to spacemacs 💜, I have squashed and cherry picked your commits into develop. @dalanicolai next time please take care that your PR has only one commit. To do so either force push on your PR branch or make single commits and squash them in the end. |
Yeah I am learning git... and sometimes some things happen that I do not understand yet... anyway I will check things again and next time think about the squash |
I suspect this PR broke the completion if not using the |
Ivy-hydra is autoloaded so the require statement here is unnecessary. Because this require (commented out) statement was here before I uncommented it in the last PR (syl20bnr#13377), I leave it here (commented out) as it was before that PR.
Ivy-hydra is autoloaded so the require statement here is unnecessary. Because this require (commented out) statement was here before I uncommented it in the last PR (syl20bnr#13377), I leave it here (commented out) as it was before that PR.
Replaced the custom ivy hydra menu with the original (and updated) ivy hydra menu. Additionally checked parts of the documentation that required modification and applied them accordingly (no modifications were needed, only some additions)