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

optimizer - allow out-of-band plugins #6177

Merged
merged 4 commits into from
Feb 12, 2016

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Feb 10, 2016

Currently plugins need to live within the Kibana source tree in order to build properly. This shouldn't be necessary, but how to accomplish this was previously unclear. Some experimentation this morning lead to a solution that seems to work great.

Now plugins can be added by simply specifying their path via the command line:

npm start -- --plugin-path=../../my-plugin

@spalger spalger changed the title [optimize] extend webpack config so plugins can live anywhere optimizer - allow out-of-band plugins Feb 10, 2016
loaderPostfixes: ['-loader', ''],
root: fromRoot('.'),
alias: this.env.aliases,
unsafeCache: this.unsafeCache,
},

resolveLoader: {
Copy link
Member

Choose a reason for hiding this comment

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

how is this added function called?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tsullivan
Copy link
Member

Looks like somethinig is choking if we have a .babelrc file in the plugin directory

without this, plugins outside of the kibana code base will need to install the babel-runtime.
@spalger
Copy link
Contributor Author

spalger commented Feb 12, 2016

yep, @tsullivan, having a .babelrc file in your plugin will cause issues unless it's designed to work with babel 5. That said, we're working on removing obstacles that were being overcome with babel/gulp and ideally we will be able to drop the .babelrc files and just use kibana's babel

@tsullivan
Copy link
Member

LGTM

@tsullivan tsullivan assigned spalger and unassigned tsullivan Feb 12, 2016
spalger added a commit that referenced this pull request Feb 12, 2016
@spalger spalger merged commit 8023a36 into elastic:master Feb 12, 2016
@spalger spalger deleted the implement/supportPluginsAnywhere branch February 25, 2016 22:49
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.

4 participants