Skip to content

Conversation

@bwnance
Copy link
Contributor

@bwnance bwnance commented Aug 26, 2022

original PR: Klipper3d/klipper#5149

@r3Fuze please let us know if you are against having your changes merged into danger-klipper!

@r3Fuze
Copy link
Contributor

r3Fuze commented Aug 26, 2022

I don't mind 👍

@chestwood96
Copy link
Contributor

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.

@bwnance
Copy link
Contributor Author

bwnance commented Sep 7, 2022

@chestwood96 it just adds support for a new do keyword to make statements clearer. Does not drop support for "do-less" statements, so no breaking changes.

@bwnance
Copy link
Contributor Author

bwnance commented Sep 7, 2022

I will add a section to the macro commands templates doc though

r3Fuze and others added 2 commits August 10, 2023 23:59
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 <r3fuze@gmail.com>
@bwnance bwnance force-pushed the r3Fuze-jinja_do_ext branch from 4d9a7f6 to b9cff9d Compare August 11, 2023 03:59
@bwnance bwnance requested a review from rogerlz August 22, 2023 19:18
@rogerlz rogerlz merged commit bad4da6 into master Aug 27, 2023
@rogerlz rogerlz deleted the r3Fuze-jinja_do_ext branch August 27, 2023 17:39
edmondhuang added a commit to edmondhuang/klipper that referenced this pull request Dec 26, 2024
1. gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
2. gcode_macro: adds RELOAD_GCODE_MACROS Klipper3d#305 KalicoCrew/kalico#305
3. python gcode templates Klipper3d#360 KalicoCrew/kalico#360
edmondhuang added a commit to edmondhuang/klipper that referenced this pull request Dec 26, 2024
1. expose math functions to gcode macros by mental Klipper3d#173 KalicoCrew/kalico#173
2. remove => gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
edmondhuang added a commit to edmondhuang/klipper that referenced this pull request Apr 29, 2025
1. gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
2. gcode_macro: adds RELOAD_GCODE_MACROS Klipper3d#305 KalicoCrew/kalico#305
3. python gcode templates Klipper3d#360 KalicoCrew/kalico#360
edmondhuang added a commit to edmondhuang/klipper that referenced this pull request Apr 29, 2025
1. expose math functions to gcode macros by mental Klipper3d#173 KalicoCrew/kalico#173
2. remove => gcode_macro: Enable jinja2.ext.do extension Klipper3d#26 KalicoCrew/kalico#26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants