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

First cut at a Rust Eventhubs client. #1715

Merged

Conversation

LarryOsterman
Copy link
Member

No description provided.

@LarryOsterman LarryOsterman marked this pull request as ready for review July 29, 2024 23:02
@LarryOsterman
Copy link
Member Author

I would appreciate any feedback on this PR - there are a couple of places I moved outside the amqp/eventhubs codebase, mostly to fix some wasm32 build issues related to tokio.

@LarryOsterman
Copy link
Member Author

@heaths, I would appreciate your feedback on this - it's very much a work in progress but it's a start.

@LarryOsterman LarryOsterman force-pushed the larryo/rust_eventhubs branch from 2c13ccd to e2bd796 Compare August 2, 2024 18:25
Copy link
Member

@heaths heaths left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went through a lot of the code and hopefully gave enough feedback for now to chew on that should permeate throughout the code. Not all is blocking, and - as usual - I've tried to preface such things with "tip" or "nit". It's late on a Friday so we can talk some more Monday morning. Overall, it's looking pretty good. Mainly just some questions about (unnecessary?) complexity and a few non-idiomatic things. I wouldn't necessarily block on those if some issue(s) were filed to keep track of changes to make later.

@LarryOsterman LarryOsterman merged commit 7fbd2b8 into Azure:feature/track2 Aug 6, 2024
16 checks passed
@LarryOsterman LarryOsterman deleted the larryo/rust_eventhubs branch August 6, 2024 21:09
@heaths heaths mentioned this pull request Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants