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 UaxEmailUrlTokenizer.cs #584

Merged
merged 1 commit into from
May 13, 2014
Merged

Update UaxEmailUrlTokenizer.cs #584

merged 1 commit into from
May 13, 2014

Conversation

jslicer
Copy link
Contributor

@jslicer jslicer commented Apr 14, 2014

removed space at the end of the Type so that it will work with ES properly.

removed space at the end of the Type so that it will work with ES properly.
@Mpdreamz
Copy link
Member

Hi @jslicer

Can you sign the elasticsearch CLA so i can pull your bits in?

http://www.elasticsearch.org/contributor-agreement/

Thanks!

@Mpdreamz
Copy link
Member

Hey @jslicer

I want to pull this in but I need you to sign the CLA before I can do so.

@jslicer
Copy link
Contributor Author

jslicer commented Apr 28, 2014

I supposedly did on the 17th.

@Mpdreamz Mpdreamz added this to the NEST 1.0-beta2 milestone May 5, 2014
@Mpdreamz
Copy link
Member

Mpdreamz commented May 5, 2014

Hi @jslicer sorry to bother you but it doesn't seem the confirmation of the CLA came through :(

Mind signing it once more?

@jslicer
Copy link
Contributor Author

jslicer commented May 5, 2014

I have signed it again and have the second signed document under cover of an email stating "Individual Contributor License Agreement (between Jesse C. Slicer and elasticsearch BV) is Signed and Filed!"

Mpdreamz added a commit that referenced this pull request May 13, 2014
Update UaxEmailUrlTokenizer.cs
@Mpdreamz Mpdreamz merged commit dd0a105 into elastic:master May 13, 2014
@Mpdreamz
Copy link
Member

Thank you @jslicer sorry for the painful CLA process, normally it doesn't take so long for CLA signatures to come in.

Looking forward to more PR's 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants