Skip to content

Make PublishAsync actually async. #112

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 18, 2025
Merged

Make PublishAsync actually async. #112

merged 1 commit into from
Mar 18, 2025
+18 −19

Conversation

lukebakken
Copy link
Contributor

Fixes #111

This required a small modification to when the TaskCompletionSource representing the operation was populated.

@lukebakken
Make PublishAsync actually async.
c3d3e20 
Fixes #111

This required a small modification to when the `TaskCompletionSource` representing the operation was populated.
@lukebakken lukebakken added this to the 0.2.0-beta.1 milestone Mar 17, 2025
@lukebakken lukebakken requested a review from Gsantomaggio March 17, 2025 22:34
@lukebakken lukebakken self-assigned this Mar 17, 2025
@lukebakken lukebakken merged commit ad3f4a4 into main Mar 18, 2025
2 checks passed
@lukebakken lukebakken deleted the rabbitmq-amqp-dotnet-client-111 branch March 18, 2025 17:17
@lukebakken
Copy link
Contributor Author

Thanks @Gsantomaggio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gsantomaggio Gsantomaggio Gsantomaggio approved these changes

Assignees

@lukebakken lukebakken

Labels
None yet
Projects
None yet
Milestone
0.2.0-rc.1
Development

Successfully merging this pull request may close these issues.

Consider to implement a callback for PublishAsync
2 participants
@lukebakken @Gsantomaggio