-
Notifications
You must be signed in to change notification settings - Fork 488
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
Enable recursion for callables in macros #1073
Conversation
docs/en/macros.md
Outdated
for n in range(count, 0, -1): | ||
yield '{n}\n'.format(n) | ||
yield n * delay_ms | ||
yield '#🎉; rm -rf /' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder what the surprise is!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest a different surprise:
curl -s -L https://raw.githubusercontent.com/keroserene/rickrollrc/master/roll.sh | bash
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is a wonderfull suggestion.
On second thought, let's maybe step back and come up with something that doesn't incite using KMK as a rubber ducky replacement or download and run potentially "untrusted" code. Something that's less edgy or malicious.
We could make it open the KMK docs in a browser tab for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is exactly the second thought I wanted to incite in a roundabout way!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason github isn't sending me any email notifications anymore.
It started out as a fix to the documentation from zulip, and here we are.