-
-
Notifications
You must be signed in to change notification settings - Fork 422
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
Allow DSL scripts for script transformation #2990
Allow DSL scripts for script transformation #2990
Conversation
Signed-off-by: Jan N. Klug <github@klug.nrw>
The current JS transformation uses |
1 similar comment
The current JS transformation uses |
Good point. I‘ll change that for the script transformation in general, since it was only merged a milestone ago or so, I think we can still accept this breaking change. I can do that this afternoon. |
… correct type Signed-off-by: Jan N. Klug <github@klug.nrw>
Signed-off-by: Jan N. Klug <github@klug.nrw>
@kaikreuzer I have changed it to |
...src/test/java/org/openhab/core/automation/module/script/ScriptTransformationServiceTest.java
Outdated
Show resolved
Hide resolved
...src/test/java/org/openhab/core/automation/module/script/ScriptTransformationServiceTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jan N. Klug <github@klug.nrw>
Done. |
@kaikreuzer Can we merge here? This should be part of 3.3 so there are less breaking changes after the release. |
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.
Sure, thank you!
* Allow DSL scripts for script transformation Signed-off-by: Jan N. Klug <github@klug.nrw> GitOrigin-RevId: 293de9d
This extends the DSL scriptengine to accept
inputString
as implicit variable in the engine context. By that DSL scripts can be used for the script transformation.Signed-off-by: Jan N. Klug github@klug.nrw