Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide AMQP 1.0 binding with fe2o3-amqp #188

Open
wants to merge 31 commits into
base: main
Choose a base branch
from

Conversation

minghuaw
Copy link
Contributor

This PR implements AMQP 1.0 binding. A working example is also provided.

Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
This reverts commit 1214976.

Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
* show feature gated bindings in docsrs

* moved crate root docsrs feature

* fixed cargo fmt check

* removed files that should go with another PR

Signed-off-by: minghuaw <[email protected]>
Signed-off-by: minghuaw <[email protected]>
@jcrossley3
Copy link
Contributor

Hey, I'm sorry I've neglected this. I'm looking to cut a new release soon. Would you mind rebasing this and let me know if you think it's "release ready"? Thanks!

@minghuaw
Copy link
Contributor Author

minghuaw commented Nov 4, 2022

Thanks for asking. There have been some big changes in fe2o3-amqp and I haven't update this PR yet. So I would not consider this as "release ready". I will update the PR in the next couple days and will let you know when I think it is ready

@jcrossley3
Copy link
Contributor

Thanks for asking. There have been some big changes in fe2o3-amqp and I haven't update this PR yet. So I would not consider this as "release ready". I will update the PR in the next couple days and will let you know when I think it is ready

Cool. I may cut one before then and we'll get you in the next one, so no rush. Thanks for your contribution and patience! 😄

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

Successfully merging this pull request may close these issues.

2 participants