-
Notifications
You must be signed in to change notification settings - Fork 77
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
gcode_macro: Enable jinja2.ext.do extension #26
Conversation
I don't mind 👍 |
Should that not be at least mentioned in the documentation somewhere? Maybe even with the nice little danger marker if/once that gets approved? While I don't know what exactly that extension does I am pretty sure it would cause makros not to be directly portable to regular klipper. |
8277760
to
ca98cff
Compare
@chestwood96 it just adds support for a new |
I will add a section to the macro commands templates doc though |
ca98cff
to
827a493
Compare
827a493
to
4d9a7f6
Compare
This adds a 'do' tag that works like a normal expression but ignores the return value. Useful for avoiding running the output of an expression as a command. Signed-off-by: Morten Lindhardt <[email protected]>
4d9a7f6
to
b9cff9d
Compare
original PR: Klipper3d/klipper#5149
@r3Fuze please let us know if you are against having your changes merged into danger-klipper!