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
Remove deprecated RenameActorReminder methods #6945
Conversation
Fixes dapr#6711 Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #6945 +/- ##
==========================================
+ Coverage 64.84% 64.95% +0.10%
==========================================
Files 228 228
Lines 20823 20731 -92
==========================================
- Hits 13503 13466 -37
+ Misses 6189 6143 -46
+ Partials 1131 1122 -9
☔ View full report in Codecov by Sentry. |
Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
/ok-to-test |
Dapr E2E testCommit ref: d1e2508 ✅ Build succeeded for linux/amd64
✅ Build succeeded for windows/amd64
✅ Infrastructure deployed
✅ Tests succeeded on windows/amd64
✅ Tests succeeded on linux/amd64
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@dapr/maintainers-dapr merging this based off of #6669 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@ItalyPaleAle Please create a tracking issue if this breaking change requires a change in SDKs as well. |
Letting #6711 remain open for any changes that are needed in SDKs. |
Fixes the issue #6711
Methods were deprecated in 1.9 and slated to be removed from 1.13.