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

[jsscripting] Update GraalJS to 21.3 to allow method selection via JavaScript #11437

Merged
merged 1 commit into from
Nov 6, 2021
Merged

Conversation

LukasA83
Copy link
Contributor

To allow method selection via javascript in case of graaljs unable to choose right method by itself, Graaljs needed to be updated to >21, e.g. for events.postUpdate with item and state when state is a DecimalType.

Signed-off-by: Lukas Agethen lukas83@gmx.de

…ript

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Oct 28, 2021
@wborn wborn changed the title [JsScript] Update Graaljs to 21.3 to allow method selection via javascript [jsscripting] Update GraalJS to 21.3 to allow method selection via JavaScript Oct 29, 2021
@LukasA83
Copy link
Contributor Author

Anything else needed for review from my side?

@digitaldan
Copy link
Contributor

I just tested this on my system and everything works as expected, so @LukasA83 the upgrade allows Graaljs to pick foo(String bar) or foo(int bar) correctly when calling from JS ?

@LukasA83
Copy link
Contributor Author

LukasA83 commented Nov 4, 2021

Well it allows you to define in case there is a conflict, see
oracle/graaljs#37

@digitaldan
Copy link
Contributor

Maybe @jpg0 can give his review, but this LGTM

Copy link
Contributor

@jpg0 jpg0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@LukasA83
Copy link
Contributor Author

LukasA83 commented Nov 6, 2021

What else is required to get this PR merged?

Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍
We only need to click a few buttons to get it merged. 😉

@wborn wborn merged commit a6fd6a3 into openhab:main Nov 6, 2021
@wborn wborn added this to the 3.2 milestone Nov 6, 2021
@wborn wborn added the enhancement An enhancement or new feature for an existing add-on label Nov 6, 2021
dschoepel pushed a commit to dschoepel/openhab-addons that referenced this pull request Nov 9, 2021
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
Signed-off-by: Dave J Schoepel <dave@theschoepels.com>
jpg0 pushed a commit to jpg0/openhab-addons that referenced this pull request Nov 10, 2021
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
mischmidt83 pushed a commit to mischmidt83/openhab-addons that referenced this pull request Jan 9, 2022
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
Signed-off-by: Michael Schmidt <mi.schmidt.83@gmail.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…vaScript (openhab#11437)

Signed-off-by: Lukas Agethen <lukas83@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants