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

Add mention about gelf.extensions.logging #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tomap
Copy link

@tomap tomap commented Feb 5, 2019

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

@tomap
Copy link
Author

tomap commented Feb 5, 2019

I do not seem to be able to sign the agreement. Page seems broken

@tomap
Copy link
Author

tomap commented Feb 6, 2019

I signed it

@jsvd jsvd requested a review from karenzone February 12, 2019 22:25
@jsvd jsvd closed this Feb 21, 2020
@jsvd jsvd reopened this Feb 21, 2020
@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
16e2132

Please, read and sign the above mentioned agreement if you want to contribute to this project

@karenzone
Copy link
Contributor

jenkins test this please

@karenzone
Copy link
Contributor

@tomap The CLA check is still failing. Sometimes this happens when the email address for the commit and the email address you used to sign the CLA are different. Please take a look and let me know.
Thanks! We are looking forward to checking out your contribution.

@karenzone
Copy link
Contributor

jenkins test this please

1 similar comment
@karenzone
Copy link
Contributor

jenkins test this please

@karenzone
Copy link
Contributor

@tomap The CLA check is still failing. Sometimes this happens when the email address for the commit and the email address you used to sign the CLA are different. Please take a look and let me know.
Thanks! We are looking forward to checking out your contribution.

@karenzone
Copy link
Contributor

jenkins test this please

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

Successfully merging this pull request may close these issues.

3 participants