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

Fix: Pass helm-org-rifle-after-init-hook as reference #77

Merged
merged 3 commits into from
Aug 21, 2023

Conversation

Thaodan
Copy link
Contributor

@Thaodan Thaodan commented Sep 15, 2022

helm-org-rifle-after-init-hook was passed as string. Helm sends a
warning when after-init-hook is not a symbol during functions such as helm-org-rifle:
Warning (emacs): Helm source ‘file.org’: after-init-hook Should be defined as a symbol

@alphapapa alphapapa self-assigned this Sep 25, 2022
@alphapapa alphapapa added this to the 1.7.2 milestone Sep 25, 2022
@alphapapa alphapapa added the bug label Sep 25, 2022
@alphapapa
Copy link
Owner

Thanks. When I have time, I'll make a new stable release with this fix.

@Rehan-MALAK
Copy link

Thanks a lot @Thaodan !

@zach-delong
Copy link

Thank you for contributing this! This seems to fix the issue on my machine. Is there anything blocking this being merged?

@Thaodan
Copy link
Contributor Author

Thaodan commented Feb 18, 2023 via email

@Thaodan
Copy link
Contributor Author

Thaodan commented Feb 18, 2023 via email

alphapapa and others added 3 commits August 21, 2023 14:21
helm-org-rifle-after-init-hook was passed as string. Helm sends a
warning when ~after-init-hook~ is not a symbol during functions such as ~helm-org-rifle~:
Warning (emacs): Helm source ‘file.org’: after-init-hook Should be defined as a symbol

Signed-off-by: Björn Bidar <[email protected]>
@alphapapa alphapapa merged commit 0e97d47 into alphapapa:master Aug 21, 2023
@alphapapa
Copy link
Owner

@Thaodan Thanks for the fix, and to you and @thierryvolpiatto for your patience!

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

Successfully merging this pull request may close these issues.

4 participants