search

dapr / python-sdk

Dapr SDK for Python
Apache License 2.0
221 stars 125 forks source link

Add the support to pass user supplied actor_factory callable in ActorRuntime & ext/fastapi,flask #714

Closed ksachdeva closed 2 months ago

ksachdeva commented 3 months ago

fixes #713

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

berndverst commented 2 months ago

Please run tox -e flake8

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 86.15%. Comparing base (fc0e9d1) to head (e5607b2). Report is 27 commits behind head on main.

Files Patch % Lines
ext/dapr-ext-fastapi/dapr/ext/fastapi/actor.py 50.00% 1 Missing :warning:
ext/flask_dapr/flask_dapr/actor.py 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #714 +/- ## ========================================== - Coverage 86.37% 86.15% -0.22% ========================================== Files 79 79 Lines 4094 4169 +75 ========================================== + Hits 3536 3592 +56 - Misses 558 577 +19 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

marcduiker commented 1 week ago

@holopin-bot @ksachdeva Thank you!

holopin-bot[bot] commented 1 week ago

Congratulations @ksachdeva, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/clzvb442509640cl6a4jfwb7f

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!