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

Support a C#-style ContinueAsNew invocation #248

Open
davidmrdavid opened this issue Feb 25, 2021 · 0 comments
Open

Support a C#-style ContinueAsNew invocation #248

davidmrdavid opened this issue Feb 25, 2021 · 0 comments
Labels
bug Something isn't working documentation Requires documentation update
Milestone

Comments

@davidmrdavid
Copy link
Collaborator

Follow-up to: Azure/azure-functions-durable-python#148

In Python we handle both the case where continueAsNew is yield'ed and the case where it is simply called. In both cases, it works as expected.

We needed to handle both cases because JS already had the precedent of requiring a yield for continueAsNew, which departs from the C# style. We should follow-up on this difference and make JS support both calling styles as well.

Follow-up Items:

  • To update the azure docs to describe this behavior, or at least to encourage the C# invocation style, for consistency
@ghost ghost added the Needs: Triage 🔍 label Feb 25, 2021
@davidmrdavid davidmrdavid self-assigned this Feb 25, 2021
@davidmrdavid davidmrdavid added bug Something isn't working documentation Requires documentation update and removed Needs: Triage 🔍 labels Feb 25, 2021
@davidmrdavid davidmrdavid added this to the 1.4.7 milestone Mar 9, 2021
@davidmrdavid davidmrdavid removed their assignment Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Requires documentation update
Projects
None yet
Development

No branches or pull requests

1 participant