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

Adds a deprecation warning for the wait() method #682

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

elena-kolevska
Copy link
Contributor

Description

Adds a deprecation warning for the wait() method

  • Code compiles correctly

Signed-off-by: Elena Kolevska <[email protected]>
@elena-kolevska elena-kolevska requested review from a team as code owners February 27, 2024 18:12
Copy link

codecov bot commented Feb 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.28%. Comparing base (fc0e9d1) to head (564aefd).
Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #682      +/-   ##
==========================================
- Coverage   86.37%   86.28%   -0.09%     
==========================================
  Files          79       82       +3     
  Lines        4094     4172      +78     
==========================================
+ Hits         3536     3600      +64     
- Misses        558      572      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@berndverst berndverst merged commit dea724d into dapr:main Mar 6, 2024
16 checks passed
berndverst pushed a commit to berndverst/python-sdk that referenced this pull request Mar 6, 2024
* Adds a deprecation warning for the wait() method

Signed-off-by: Elena Kolevska <[email protected]>

* Ruff

Signed-off-by: Elena Kolevska <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>
berndverst added a commit that referenced this pull request Mar 6, 2024
* Adds a deprecation warning for the wait() method (#682)

* Adds a deprecation warning for the wait() method

Signed-off-by: Elena Kolevska <[email protected]>

* Ruff

Signed-off-by: Elena Kolevska <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>

* Release Python SDK 1.13.0

Signed-off-by: Bernd Verst <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>
Signed-off-by: Bernd Verst <[email protected]>
Co-authored-by: Elena Kolevska <[email protected]>
elena-kolevska added a commit to elena-kolevska/python-sdk that referenced this pull request Apr 24, 2024
* Adds a deprecation warning for the wait() method

Signed-off-by: Elena Kolevska <[email protected]>

* Ruff

Signed-off-by: Elena Kolevska <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>
berndverst added a commit that referenced this pull request Jun 27, 2024
* Brings back the link we removed because it wasn’t online yet

Signed-off-by: Elena Kolevska <[email protected]>

* Retry decorator

Signed-off-by: Elena Kolevska <[email protected]>

* Use a retry function instead of a decorator

Signed-off-by: Elena Kolevska <[email protected]>

* adds retries for async

Signed-off-by: Elena Kolevska <[email protected]>

* Fixes types

Signed-off-by: Elena Kolevska <[email protected]>

* Timeout policy

Signed-off-by: Elena Kolevska <[email protected]>

* Cleanup

Signed-off-by: Elena Kolevska <[email protected]>

* retry for http wip.

Signed-off-by: Elena Kolevska <[email protected]>

* tests for http retries

Signed-off-by: Elena Kolevska <[email protected]>

* New ruff version has some formatting changes

Signed-off-by: Elena Kolevska <[email protected]>

* Revert "New ruff version has some formatting changes"

This reverts commit 8609b7f.

Signed-off-by: Elena Kolevska <[email protected]>

* Fixes setting values with a default of None

Signed-off-by: Elena Kolevska <[email protected]>

* Moves timeout

Signed-off-by: Elena Kolevska <[email protected]>

* Adds a deprecation warning for the wait() method (#682)

* Adds a deprecation warning for the wait() method

Signed-off-by: Elena Kolevska <[email protected]>

* Ruff

Signed-off-by: Elena Kolevska <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>

* Pin ruff version

Signed-off-by: Bernd Verst <[email protected]>
Signed-off-by: Elena Kolevska <[email protected]>

* Pin ruff version in tox.ini

Signed-off-by: Bernd Verst <[email protected]>
Signed-off-by: Elena Kolevska <[email protected]>

* Lint fix

Signed-off-by: Elena Kolevska <[email protected]>

* Reorganises tests. Adds retry for actors.

Signed-off-by: Elena Kolevska <[email protected]>

* Linters

Signed-off-by: Elena Kolevska <[email protected]>

* Retry all calls

Signed-off-by: Elena Kolevska <[email protected]>

* Small fix and docs update

Signed-off-by: Elena Kolevska <[email protected]>

* Adds a note to docs, for clarity

Signed-off-by: Elena Kolevska <[email protected]>

---------

Signed-off-by: Elena Kolevska <[email protected]>
Signed-off-by: Bernd Verst <[email protected]>
Co-authored-by: Bernd Verst <[email protected]>
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