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

remove elasticsearch- from name of official plugins #12158

Merged
merged 1 commit into from
Jul 9, 2015

Conversation

jaymode
Copy link
Member

@jaymode jaymode commented Jul 9, 2015

This change fixes the plugin manager to trim elasticsearch- and es- prefixes from plugin names
for our official plugins. This restores the old behavior prior to #11805.

Closes #12143

This change fixes the plugin manager to trim `elasticsearch-` and `es-` prefixes from plugin names
for our official plugins. This restores the old behavior prior to elastic#11805.

Closes elastic#12143
@rmuir
Copy link
Contributor

rmuir commented Jul 9, 2015

Looks good to me.

@dadoonet
Copy link
Member

dadoonet commented Jul 9, 2015

Thank you @jaymode!

jaymode added a commit that referenced this pull request Jul 9, 2015
remove elasticsearch- from name of official plugins
@jaymode jaymode merged commit ef4c255 into elastic:master Jul 9, 2015
@kevinkluge kevinkluge removed the review label Jul 9, 2015
jaymode added a commit to jaymode/elasticsearch that referenced this pull request Jul 9, 2015
elastic#12158 only partially fixed elastic#12143 by removing the prefix from official plugin names. This change
removes the prefixes from any plugin names.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Core/Infra/Plugins Plugin API and infrastructure v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recent changes to Pluginmanager broke naming convention
4 participants