Skip to content

[Actors API] update err desc to be more explicit - #643

Merged
yaron2 merged 4 commits into
dapr:mainfrom
cicoyle:fix-improve-actor-api-experience
Dec 15, 2025
Merged

yaron2 merged 4 commits into
dapr:mainfrom
cicoyle:fix-improve-actor-api-experience

Conversation

@cicoyle

@cicoyle cicoyle commented Oct 23, 2024

Copy link
Copy Markdown
Contributor

Ran into some errors with using the Actors API & reminders. This PR is a nice to have making it explicit to newbie actor users what the error means.

This PR fixes one of the issues, which I also had locally and confirm resolves the err:

== APP == 2024/10/22 13:54:53 http: superfluous response.WriteHeader call from [github.com/dapr/go-sdk/service/http.(*Server).registerBaseHandler.func6](http://github.com/dapr/go-sdk/service/http.(*Server).registerBaseHandler.func6) (topic.go:214)

Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
@cicoyle
cicoyle requested review from a team as code owners October 23, 2024 14:40
@codecov

codecov Bot commented Oct 23, 2024 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 55.79%. Comparing base (6c59092) to head (d1d687c).
⚠️ Report is 37 commits behind head on main.

Files with missing lines Patch % Lines
actor/manager/manager.go 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (6c59092) and HEAD (d1d687c). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (6c59092) HEAD (d1d687c)
4 1
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #643      +/-   ##
==========================================
- Coverage   62.52%   55.79%   -6.74%     
==========================================
  Files          56       58       +2     
  Lines        4139     3692     -447     
==========================================
- Hits         2588     2060     -528     
- Misses       1425     1496      +71     
- Partials      126      136      +10     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread actor/manager/manager.go Outdated
Co-authored-by: Mike Nguyen <hey@mike.ee>
Signed-off-by: Cassie Coyle <cassie.i.coyle@gmail.com>

@mikeee mikeee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cicoyle

cicoyle commented Dec 11, 2025

Copy link
Copy Markdown
Contributor Author

I am simply changing the text in the errors.New() but nothing else, can this be merged even with the codecov failure since Im just trying to make the error make sense to users when they see it bc as-is, it makes no sense.

@yaron2
yaron2 merged commit 4c5a460 into dapr:main Dec 15, 2025
18 of 21 checks passed
@cicoyle cicoyle added this to the v1.14 milestone Jan 16, 2026
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.

5 participants