Added get_nodes and a couple pep8 fixes

bkjones / pyrabbit

A Python module to communicate w/ the RabbitMQ HTTP Management API

http://pyrabbit.readthedocs.org

BSD 3-Clause "New" or "Revised" License

97 stars 86 forks source link

Added get_nodes and a couple pep8 fixes #15

Closed spil-sean closed 10 years ago

spil-sean commented 11 years ago

I wanted to get the stats for a node out with pyrabbit and it was missing get_nodes() so I added it.

spil-sean commented 11 years ago

ping...

spil-sean commented 10 years ago

ping?

bkjones commented 10 years ago

Sorry for the delayed reply @spil-sean - thanks for the added api coverage, but I do require tests for code you're adding before I'll do a merge. It should take you all of 5 minutes to do following the patterns laid out in the existing test modules. It'll probably look something like: https://github.com/bkjones/pyrabbit/blob/master/tests/test_pyrabbit.py#L40

spil-sean commented 10 years ago

I'll make sure to add tests if I add anything in the future. Thanks.

bkjones commented 10 years ago

Thanks again, @spil-sean