streadway/amqp — pull request #12 by peterbourgon (2012-07-06 15:33:02+00:00)
Title: Development continues

I did something which allowed `(defer) connection.Close()` to succeed in the integration tests. Long story short, Close() was waiting for a connectionCloseOk response, which was being gobbled by dispatch(). I propagated the message to Close(), which seemed to have the desired effect, but it could be that the expectations of Close() are incorrect; my understanding of the 0.9.1 close handshake is still shaky.

In any case, it seems like (subjectively) too many things are responsible for/capable of triggering a shutdown() on the connection. I would like to take another (probably full) day to refactor that. Any tips or direction you wanted to provide would be appreciated. I'll file an issue for myself for that.

I also updated the README a bit, and added a travis.yml. The latter will only work once you merge to master, which I would personally vote to do ASAP. When that happens I can make a small announcement on the mailing list, and hopefully get some more minds applied to making the lib even more ~~Awesome~~ Remarkable.