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

Its Not clear which is the most up-to-date-dotnet Workflow Sample #981

Closed
tshaiman opened this issue Feb 9, 2024 · 2 comments
Closed

Its Not clear which is the most up-to-date-dotnet Workflow Sample #981

tshaiman opened this issue Feb 9, 2024 · 2 comments
Assignees
Milestone

Comments

@tshaiman
Copy link

tshaiman commented Feb 9, 2024

There are two versions of .NET SDK Workflow samples , in two different repos :

https://github.com/dapr/quickstarts/tree/master/workflows/csharp/sdk/order-processor

and

https://github.com/dapr/dotnet-sdk/tree/master/examples/Workflow/WorkflowConsoleApp

which is the "Lastest-And-Greatest" ?

@paulyuk
Copy link
Contributor

paulyuk commented Feb 11, 2024

Hi @tshaiman

In principle quickstarts are meant to be the most official, best supported, most looked at samples. They always cover all the languages. SDK authors will also include examples as they author. Sometimes there are brief moments where SDK examples are "ahead" while authors are improving their SDKs, but I also insist we get latest and greatest back into quickstarts.

@halspang @philliphoff @MregXN fyi.

Now, this PR from @MregXN has been languishing until today so the quickstarts where stale showing which workflow client to use. I just merged that to correct. Thanks for pointing this out.

All good?

@paulyuk paulyuk self-assigned this Feb 11, 2024
@paulyuk paulyuk added this to the 1.13 milestone Feb 11, 2024
@tshaiman
Copy link
Author

@paulyuk absolutely !
thanks for that .

also , I will be more then happy to perform a full K8s Tutorial. I know there is "Hello Kubernetes" but I'm talking about full End-to-End.
( also in my opinion using the Dapr CLI in k8s scenarios is not applicable , as all should be deployed with Yaml / Helm / Kustomize and not with dedicated cli , as cool as it may be . )

keep on rocking !

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

No branches or pull requests

3 participants