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

update index pattern convention #39

Closed
wants to merge 23 commits into from
Closed

update index pattern convention #39

wants to merge 23 commits into from

Conversation

lmangani
Copy link
Contributor

@lmangani lmangani commented Dec 1, 2015

Patch proposal for testing (missing data to fully validate each index) addressing #36 and minor user req. #38

@tsg
Copy link
Contributor

tsg commented Dec 1, 2015

Thanks @lmangani, will check it tomorrow.

@tsg tsg added the review label Dec 1, 2015
@monicasarbu
Copy link
Contributor

@tsg I can take care of this PR.

@lmangani
Copy link
Contributor Author

lmangani commented Dec 1, 2015

Fixed recurring pcre-fu glitches

@ruflin
Copy link
Contributor

ruflin commented Dec 2, 2015

@lmangani Could you squash the commits?

@lmangani
Copy link
Contributor Author

lmangani commented Dec 2, 2015

ACK

@ruflin
Copy link
Contributor

ruflin commented Dec 2, 2015

@lmangani As I saw you merged in master, not sure if there was a misunderstanding. As you have made lots of small commits and some fixes I wanted to ask if you feel comfortable to squash the commits into 1 single commit?

@lmangani
Copy link
Contributor Author

lmangani commented Dec 2, 2015

Let me close and re-PR clean

@lmangani lmangani closed this Dec 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants