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

feat: add SendWithContext #1

Merged
merged 2 commits into from
Sep 27, 2022
Merged

feat: add SendWithContext #1

merged 2 commits into from
Sep 27, 2022

Conversation

jonlundy
Copy link
Contributor

No description provided.

@proofrock proofrock self-assigned this Sep 25, 2022
@proofrock
Copy link
Owner

Hi Jon, sorry for the late reply - thank you so much for this! In the next days I'll review it. Could you kindly maybe add a test case? Thanks again!

@proofrock proofrock added the enhancement New feature or request label Sep 25, 2022
@jonlundy
Copy link
Contributor Author

@proofrock hey no worries! I have added a test case for a cancelled context.

@jonlundy jonlundy force-pushed the context branch 2 times, most recently from f90c62e to ffd2612 Compare September 26, 2022 15:35
@proofrock
Copy link
Owner

Excellent! Very nice, thanks a lot. Now I only have to figure out how to apply this to the develop branch...

@jonlundy jonlundy changed the base branch from main to develop September 27, 2022 16:25
@jonlundy
Copy link
Contributor Author

Oh! i can switch to that one

@proofrock
Copy link
Owner

That would be great. If you can't, I'll try to rebase on develop, but the risk is that GitHub doesn't show your contribution on the project page... :-(

@jonlundy
Copy link
Contributor Author

@proofrock done. cherry-picked over to develop

@proofrock proofrock merged commit 2279538 into proofrock:develop Sep 27, 2022
@proofrock
Copy link
Owner

Ok, done. Thanks a whole lot!!

@proofrock proofrock self-requested a review September 27, 2022 16:38
@proofrock
Copy link
Owner

Tested and released. Again, many many thanks!

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

Successfully merging this pull request may close these issues.

2 participants