search

logstash-plugins / logstash-input-jms

12 stars 27 forks source link

There is no pub-sub, it is pub_sub #34

Closed DanRoscigno closed 5 years ago

DanRoscigno commented 5 years ago

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

DanRoscigno commented 5 years ago

With the new PR from @robbavey this is no longer current. But, in your docs Rob you have a mix of pub-sub and pub_sub. I would think that they should all be pub_sub.

robbavey commented 5 years ago

@DanRoscigno You are absolutely correct - I've pushed a change to fix this in my PR