-
Notifications
You must be signed in to change notification settings - Fork 841
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
EUNSUPPORTEDPROTOCOL #1239
Comments
The lib is installed well if I manually add |
The EUI library does not support being installed via |
@chandlerprall but there is this instruction in the readme https://github.com/elastic/eui#installation
|
I'll update that doc, thanks! |
I am using EUI for a plugin that is compatible with Kibana 6.1.x. Now I assume EUI (from 4.3.0 upwards, from my tests) can no longer be used to write plugins for Kibana versions lower than 6.2 (I recall that's when |
You're correct, yarn was introduced in Kibana 6.2 (elastic/kibana#15485). If your plugin authoring process requires EUI to be installed/managed by Kibana, you would need to stick to a pre-4.3.0 EUI. However, if you're able to build your plugin code outside of Kibana you should be able to use any version of EUI. |
Just for the sake of discussion, I'm curious about the decision to only support installation via Background: I'm playing with using EUI to manage layout and default styles for a non-Kibana UI that Infra is working on and stumbled around for a little while trying to get it installed before realizing the docs said it was Yarn-only. I'd love to help work on adding npm support, unless I'm the only person who thinks the effort would be worth it. 🙂 |
We're using yarn's ability to alias packages to use Elastic/Kibana's version of lodash@3.10.1, which contains a security fix. This is meant as temporary (though low priority) until we either migrate to lodash 4 or off lodash entirely ( #360 ) |
I got the following error when trying to install the lib
In my plugin I use lodash v4.17.10.
The text was updated successfully, but these errors were encountered: