Skip to content

Conversation

@jdconrad
Copy link
Contributor

With the upcoming instance bindings, the singular *Binding name isn't descriptive enough with multiple binding types. This renames the existing *Binding classes to *ClassBinding. Mechanical change (with some error messages changed from binding to class binding by hand).

@jdconrad jdconrad added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache v7.0.0 >refactoring v6.5.0 labels Sep 19, 2018
@jdconrad jdconrad requested a review from rjernst September 19, 2018 15:40
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jdconrad jdconrad mentioned this pull request Sep 19, 2018
23 tasks
Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@jdconrad
Copy link
Contributor Author

@rjernst Thanks for the review!

@jdconrad jdconrad merged commit 3df285d into elastic:master Sep 19, 2018
jdconrad added a commit that referenced this pull request Sep 19, 2018
With the upcoming instance bindings, the singular *Binding name isn't descriptive enough 
with multiple binding types. This renames the existing *Binding classes to *ClassBinding. 
Mechanical change (with some error messages changed from binding to class binding by 
hand).
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 >refactoring v6.5.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants