Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

Add support for amqplib #156

Merged
merged 7 commits into from
Sep 13, 2016
Merged

Add support for amqplib #156

merged 7 commits into from
Sep 13, 2016

Conversation

phucpnt
Copy link
Contributor

@phucpnt phucpnt commented Sep 7, 2016

Hello,

I hope this feature would be put into mainstream.

Regards
Phuc

@mcollina
Copy link
Collaborator

mcollina commented Sep 8, 2016

Can you add this to the README?

@mcollina
Copy link
Collaborator

mcollina commented Sep 8, 2016

all unit tests are failing https://travis-ci.org/mcollina/ascoltatori/jobs/158252830.

@phucpnt
Copy link
Contributor Author

phucpnt commented Sep 8, 2016

@mcollina I saw it, will do the update and let you know

@phucpnt
Copy link
Contributor Author

phucpnt commented Sep 9, 2016

@mcollina I has updated the PR. The readme also updated. I check the test and it passed

any concern to merge the pr?

@mcollina
Copy link
Collaborator

@phucpnt there are still lots of console.log in the PR https://travis-ci.org/mcollina/ascoltatori/jobs/158910023#L1314.

@phucpnt
Copy link
Contributor Author

phucpnt commented Sep 12, 2016

@mcollina my bad, I has remove the console.log

@mcollina mcollina merged commit eb13909 into moscajs:master Sep 13, 2016
@mcollina
Copy link
Collaborator

Released as v3.1.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants