Skip to content

Conversation

costin
Copy link
Member

@costin costin commented Nov 23, 2018

Remove unused grammar declaration and minor polishing inside parser.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@costin costin requested review from astefan and matriv November 23, 2018 12:06
@costin costin force-pushed the interval-grammar-polish branch from 2d0bdc7 to 08ee214 Compare November 23, 2018 12:06
Copy link
Contributor

@matriv matriv left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@astefan astefan left a comment

Choose a reason for hiding this comment

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

LGTM

@costin
Copy link
Member Author

costin commented Nov 23, 2018

Thanks

@costin costin merged commit e22a835 into elastic:master Nov 23, 2018
@costin costin deleted the interval-grammar-polish branch November 23, 2018 15:14
costin added a commit that referenced this pull request Nov 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment