Results 584 comments of Ben Abrams

I'd prefer we disable it inline and link above it with a `TODO: ` to remove it later. More specific disabling is generally better than blanket disabling it. It leaves...

I have been disabling it inline for the gemspec as there is no value there, there is good reason to keep blocks relatively short as they are basically being passed...

I think we probably want to maybe tweak the default to say enable it but limit it to 50 (opposed to the 25) lines in a block. Honestly if you...

If no one claims these first in a couple sprints my team will knock out any of the ones we use and are not already done: - sensu-plugin - ~~sensu-plugins-aws~~...

I don't have a lot of experience with golang development but let me know when you want to get together to discuss it so that I can learn more about...

@eheydrick I have not seen them but we are gonna be bumping everything to 2.x shortly anyways so it seems like a moot point to update both and then update...

@portertech thanks for the help, please do them as PRs with your travis changes so that we can keep track of them all here and avoid committing to master directly....

@portertech for sure I will review them shortly I will probably merge and release the ones that have breaking changes tonight as I don't believe in functional changes not being...

@Evesy @thomasriley you guys are awesome! We are over 1/2 way done with install testing. There is no way I would have gotten around to all this on my own...

rethinkdb has had changes but is blocked on release because of this: https://github.com/sensu-plugins/sensu-plugins-rethinkdb/issues/2