Conversation
Codecov Report
@@ Coverage Diff @@
## master #758 +/- ##
=======================================
Coverage 68.90% 68.90%
=======================================
Files 137 137
Lines 4458 4458
Branches 500 500
=======================================
Hits 3072 3072
Misses 1271 1271
Partials 115 115
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
| On Windows: set DAPR_HTTP_PORT=<port> | ||
| On Linux, MacOS: export DAPR_HTTP_PORT=<port> | ||
| ``` | ||
| dapr run --app-id actor_client dotnet run |
There was a problem hiding this comment.
What's the reason for doing this? You don't need to run this app with a dapr sidecar to invoke the actors. I think showing that you can interact with actors from outside of dapr is an important point.
There was a problem hiding this comment.
Then both options should be added to the docs to explain the two ways.
In real scenarios, even if you are communicating with actors from the outside - this "outside" application will not run in the actors cluster - thus expecting dapr sidecar of the actors app is running on the same machine will not be possible.
Description
Issue reference
No - only small changes
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: