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

Fix optional default script loading #6582

Closed
wants to merge 1 commit into from

Conversation

kimchy
Copy link
Member

@kimchy kimchy commented Jun 20, 2014

Groovy is optional as a dependency in the classpath, make sure we properly detect when its not at the right time to disable it

Groovy is optional as a dependency in the classpath, make sure we properly detect when its not at the right time to disable it
closes elastic#6582
@dakrone
Copy link
Member

dakrone commented Jun 20, 2014

LGTM, +1

@kimchy kimchy closed this in 0d66d37 Jun 20, 2014
kimchy added a commit that referenced this pull request Jun 20, 2014
Groovy is optional as a dependency in the classpath, make sure we properly detect when its not at the right time to disable it
closes #6582
@kimchy kimchy deleted the fix_script_cl branch June 20, 2014 22:31
@clintongormley clintongormley changed the title Fix optional default script loading Scripting: Fix optional default script loading Jul 16, 2014
@clintongormley clintongormley added the :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache label Jun 7, 2015
@clintongormley clintongormley changed the title Scripting: Fix optional default script loading Fix optional default script loading Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >enhancement v1.3.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants