majek / puka

Puka - the opinionated RabbitMQ client
https://github.com/majek/puka
Other
182 stars 34 forks source link

puka.html claims support for immediate arg, but code doesn't support it #61

Closed vitaly-krugl closed 9 years ago

vitaly-krugl commented 9 years ago

FYI:

Per http://majek.github.io/puka/puka.html: Client.basic_publish(exchange, routing_key, mandatory=False, immediate=False, headers={}, body="").

But the code in https://github.com/majek/puka/blob/master/puka/machine.py doesn't have support for immediate: def basic_publish(conn, exchange, routing_key='', mandatory=False, headers={}, body='')

majek commented 9 years ago

https://github.com/majek/puka/commit/b5699c4ffdeeb1264a0a8a99b4221e945cc71057

Rabbitmq 3.0 doesn't do it I guess?

I guess we could update puka.html....

vitaly-krugl commented 9 years ago

Right, I meant that puka.html is out of sync.