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

Initial commit #1

Merged
merged 1 commit into from
Jan 18, 2017
Merged

Initial commit #1

merged 1 commit into from
Jan 18, 2017

Conversation

erjohnso
Copy link
Contributor

@erjohnso erjohnso commented Nov 7, 2016

Hi @suyograo,

Sorry for the delay and thanks for creating the repo. Here's the initial PR to add a new input plugin for Google PubSub.

I'm eager to help resolve any issues you find with the plugin.

Cheers, Eric

@erjohnso
Copy link
Contributor Author

Hello @suyograo - anything else you need from me to help with the review / merge?

1 similar comment
@erjohnso
Copy link
Contributor Author

erjohnso commented Dec 2, 2016

Hello @suyograo - anything else you need from me to help with the review / merge?

@ahmerbasit
Copy link

Hi @suyograo, do we have an ETA on when this might get reviewed/submitted/merged?

@ahmerbasit
Copy link

Any update?

@ahmerbasit
Copy link

I am struggling to find out what's the next step here. There is virtually no response from suygrao@ in over a month.

@acchen97
Copy link

Thanks for being patient with us. The team is currently heads down crunching out the next minor release right now, we'll circle back sometime after that.

Appreciate the interest and the PR @erjohnso

@moeadham
Copy link

So, is anybody going to merge this?

@suyograo suyograo merged commit 5260992 into logstash-plugins:master Jan 18, 2017
@acchen97
Copy link

@erjohnso thanks for your contribution, its an excellent addition to the ecosystem. I'll be reaching out to you directly regarding continued maintenance of this plugin.

@erjohnso
Copy link
Contributor Author

sg, thanks @acchen97 and @suyograo!

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.

5 participants