-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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] Upgrade openhab-js & Remove SharedCache #13908
[jsscripting] Upgrade openhab-js & Remove SharedCache #13908
Conversation
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
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.
It seems to break the build:
Error: Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.12.1:npx (npx webpack) on project org.openhab.automation.jsscripting: Failed to run task: 'npx webpack -c ./node_modules/openhab/webpack.config.js --entry ./node_modules/openhab/ -o ./dist' failed. org.apache.commons.exec.ExecuteException: Process exited with an error: 2 (Exit value: 2) -> [Help 1]
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
I see the problem, I'll fix it later. |
The addon build failed due to webpack exiting with 1 because of a missing webpack configuration for „@runtime/cache“. Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@wborn The build is now working. I‘ve also had a look at the classed in the |
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.
Thanks! 👍 It's nice to also have this available in M6 as well.
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Upgrades the included openhab-js version to 3.1.0, which uses the new caches from core (introduced in openhab/openhab-core#2887) and provides many doc improvements. Removes the SharedCache from the addon because this functionality is now provided by core (see openhab/openhab-core#2887). Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
This PR upgrades the included openhab-js version to 3.1.0, which uses the new caches from core instead of the addon provided one and brings many doc improvements.
The
SharedCache
is therefore removed as it is superseded by the caches added to core in openhab/openhab-core#2887.@jlaur Ping.
@wborn FYI: We can now remove the
SharedCache
.