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

Properly handle SynchronizatonContext #31

Merged
merged 2 commits into from
Dec 2, 2023
Merged

Properly handle SynchronizatonContext #31

merged 2 commits into from
Dec 2, 2023

Conversation

murshex
Copy link
Contributor

@murshex murshex commented Dec 2, 2023

:)

@MarkCiliaVincenti
Copy link
Owner

Thank you for your contribution :)

Can you carry out these changes please?

  1. There is no need to create new methods; just use the existing ones and add the parameter with a default value of false.
  2. Please change the summary description to: "true to attempt to marshal the continuation back to the original context captured; otherwise, false. Defaults to false."

@murshex
Copy link
Contributor Author

murshex commented Dec 2, 2023

Modified as per request of Mr. Vincenti

@MarkCiliaVincenti
Copy link
Owner

Thank you! I'll release as beta until this new functionality is covered by tests.

@MarkCiliaVincenti MarkCiliaVincenti merged commit 1819af4 into MarkCiliaVincenti:master Dec 2, 2023
3 checks passed
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